Change parameter types to use move semantic 34/191434/1
authorJunghoon Park <jh9216.park@samsung.com>
Wed, 17 Oct 2018 06:38:43 +0000 (15:38 +0900)
committerJunghoon Park <jh9216.park@samsung.com>
Wed, 17 Oct 2018 06:38:43 +0000 (15:38 +0900)
Change-Id: Ie3788d5e616353640b47c150941139198453897f
Signed-off-by: Junghoon Park <jh9216.park@samsung.com>
src/proxy-internal.cc
src/proxy-internal.h

index 4830f428508c2c89553827f782272bbba889a165..262634b00716277d39dcdaf27c35ba8c884cd13d 100644 (file)
@@ -124,8 +124,8 @@ void Proxy::OnPortVanished(const std::string& appid,
       appid.c_str(), port_name.c_str());
 }
 
-int Proxy::Connect(const std::string appid, const std::string& port_name,
-                    IEventListener* ev) {
+int Proxy::Connect(std::string appid, std::string port_name,
+                   IEventListener* ev) {
   if (ev == nullptr)
     return RPC_PORT_ERROR_INVALID_PARAMETER;
 
@@ -135,9 +135,9 @@ int Proxy::Connect(const std::string appid, const std::string& port_name,
   }
 
   listener_ = ev;
-  target_appid_ = appid;
-  port_name_ = port_name;
-  int r = fd_broker_.Watch(this, appid, port_name);
+  target_appid_ = std::move(appid);
+  port_name_ = std::move(port_name);
+  int r = fd_broker_.Watch(this, target_appid_, port_name_);
   if (r < 0) {
     listener_ = nullptr;
     if (r == -EILLEGALACCESS)
index f5768ed184eb069b21c31d8a0d8622803b22b3ba..8584ca269f7f4915163c2131b2c01cfc0ab28042 100644 (file)
@@ -43,11 +43,12 @@ class Proxy : public FdBroker::IEventWatcher {
     virtual void OnReceived(const std::string& endpoint) = 0;
   };
 
-  int Connect(const std::string appid, const std::string& port_name,
-               IEventListener* ev);
+  int Connect(std::string appid, std::string port_name, IEventListener* ev);
+
   std::shared_ptr<Port> GetPort() const {
     return main_port_;
   }
+
   std::shared_ptr<Port> GetDelegatePort() const {
     return delegate_port_;
   }