Fix the implementation related to rpc-port

- Removes unnecessary log prints
- Fixes the wrong implementation about rpc_port_parcel_header_h.

Change-Id: I0a9676c1823a372d61fb30edaab8935490780de4
Signed-off-by: Hwankyu Jhun <h.jhun@samsung.com>

diff --git a/idlc/gen/c_proxy_body_gen_cb.h b/idlc/gen/c_proxy_body_gen_cb.h
index 51427397de253328695a6b20559fe45ad2f5a568..780e4b52880e76e2d33274ab83bcc983d95e5c98 100644 (file)
--- a/idlc/gen/c_proxy_body_gen_cb.h
+++ b/idlc/gen/c_proxy_body_gen_cb.h
@@ -764,7 +764,6 @@ void <PREFIX>_<NAME>_invoke_<METHOD_NAME>(<PREFIX>_<NAME>_h h<METHOD_PARAMS>)
   rpc_port_parcel_get_header(parcel_, &header_);
   rpc_port_parcel_header_set_tag(header_, TIDL_VERSION);
   rpc_port_parcel_header_get_seq_num(header_, &seq_num_);
-  _W("[Version] \"%s\", [Sequence] %d", TIDL_VERSION, seq_num_);
 
   rpc_port_parcel_write_int32(parcel_, <UPPERCASE_PREFIX>_<UPPERCASE_NAME>_METHOD_<UPPERCASE_METHOD_NAME>);
 
@@ -836,7 +835,6 @@ R"__c_cb(
   rpc_port_parcel_get_header(parcel_, &header_);
   rpc_port_parcel_header_set_tag(header_, TIDL_VERSION);
   rpc_port_parcel_header_get_seq_num(header_, &seq_num_);
-  _W("[Version] \"%s\", [Sequence] %d", TIDL_VERSION, seq_num_);
 
   rpc_port_parcel_write_int32(parcel_, <UPPERCASE_PREFIX>_<UPPERCASE_NAME>_METHOD_<UPPERCASE_METHOD_NAME>);
 
@@ -866,7 +864,6 @@ R"__c_cb(
     tag_ = nullptr;
     rpc_port_parcel_header_get_tag(header_, &tag_);
     if (tag_ && tag_[0] != '\0') {
-      _W("[Version] %s", tag_);
       rpc_port_parcel_header_get_seq_num(header_, &recv_seq_num_);
       if (recv_seq_num_ != seq_num_) {
         _E("Invalid protocol. %d", recv_seq_num_);

diff --git a/idlc/gen/c_stub_body_gen_cb.h b/idlc/gen/c_stub_body_gen_cb.h
index 6c63fb59abf470d84f90b15976b0efded0d2d194..aafe2f1ca33f6083c2ba4a650232a1d1aa7f8cd6 100644 (file)
--- a/idlc/gen/c_stub_body_gen_cb.h
+++ b/idlc/gen/c_stub_body_gen_cb.h
@@ -715,7 +715,6 @@ static int __<PREFIX>_<NAME>_method_<METHOD_NAME>_handler(rpc_port_h port, rpc_p
 
   rpc_port_parcel_get_header(parcel, &header_);
   rpc_port_parcel_header_get_seq_num(header_, &seq_num_);
-  _W("[Sequence] %d", seq_num_);
 
   <METHOD_HANDLER_PARCEL_READ>
   <METHOD_HANDLER_CALLBACK_INVOKE>

diff --git a/idlc/gen/cpp_proxy_body_gen_cb.h b/idlc/gen/cpp_proxy_body_gen_cb.h
index d81ebb189d0e319919b21e0d3198a50f94bb6ff2..58e870a4852dbfdfce16b2f64368b6ebf9985f8d 100644 (file)
--- a/idlc/gen/cpp_proxy_body_gen_cb.h
+++ b/idlc/gen/cpp_proxy_body_gen_cb.h
@@ -41,7 +41,6 @@ R"__cpp_cb(
   rpc_port_parcel_header_set_tag(header_, TIDL_VERSION);
   int seq_num_ = -1;
   rpc_port_parcel_header_get_seq_num(header_, &seq_num_);
-  _W("[Version] \"%s\", [Sequence] %d", TIDL_VERSION, seq_num_);
   rpc_port_parcel_write_int32(p, static_cast<int>(MethodId::<METHOD>));
 <SERIALIZE>
   // Send
@@ -78,7 +77,6 @@ R"__cpp_cb(
     rpc_port_parcel_header_get_tag(header_, &tag_);
     std::unique_ptr<char, decltype(std::free)*> tag_auto_(tag_, std::free);
     if (tag_ && tag_[0] != '\0') {
-      _W("[Version] %s", tag_);
       int seq_num_received_ = -1;
       rpc_port_parcel_header_get_seq_num(header_, &seq_num_received_);
       if (seq_num_received_ != seq_num_) {

diff --git a/idlc/gen/cpp_stub_body_gen.cc b/idlc/gen/cpp_stub_body_gen.cc
index 455785d3f87ea863581b4a428b174445b691c412..2c51d1f4d0f0ff95c8f9abce7522c669af519507 100644 (file)
--- a/idlc/gen/cpp_stub_body_gen.cc
+++ b/idlc/gen/cpp_stub_body_gen.cc
@@ -242,7 +242,6 @@ void CppStubBodyGen::GenInvocation(std::ofstream& stream,
   }
 
   m += "ret = rpc_port_parcel_send(result, port);\n";
-  m += "_I(\"Parcel send result(%d)\", ret);\n";
   stream << AddIndent(TAB_SIZE * 3, m);
 }
 

diff --git a/idlc/gen/cpp_stub_body_gen_cb.h b/idlc/gen/cpp_stub_body_gen_cb.h
index 3bcac51e16d4f135ce38596f88e73c8c60dd7800..c88e6599ba6bea46079f14e6818a9f471df88830 100644 (file)
--- a/idlc/gen/cpp_stub_body_gen_cb.h
+++ b/idlc/gen/cpp_stub_body_gen_cb.h
@@ -119,7 +119,7 @@ int [NAME]::OnReceivedCB(const char* sender, const char* instance, rpc_port_h po
   auto* cxt = static_cast<[NAME]*>(data);
   rpc_port_parcel_h p;
   rpc_port_parcel_h result;
-  rpc_port_parcel_h header;
+  rpc_port_parcel_header_h header;
   int seq_num = -1;
   int cmd;
   int ret;
@@ -152,7 +152,6 @@ int [NAME]::OnReceivedCB(const char* sender, const char* instance, rpc_port_h po
 
   rpc_port_parcel_get_header(p, &header);
   rpc_port_parcel_header_get_seq_num(header, &seq_num);
-  _W("[Sequence] %d", seq_num);
 
   rpc_port_parcel_create(&result);
   rpc_port_parcel_get_header(result, &header);