Fix setting callback to use move operation in PrsenceSubscriber
authorcoderhyme <jhyo.kim@samsung.com>
Sat, 4 Jul 2015 02:46:49 +0000 (11:46 +0900)
committerUze Choi <uzchoi@samsung.com>
Mon, 6 Jul 2015 01:32:46 +0000 (01:32 +0000)
It was copying callback which is unneccesary overhead.

Change-Id: Ic867a1e68225a95b2537737c9290ec0d3fe6354c
Signed-off-by: coderhyme <jhyo.kim@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/1516
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Madan Lanka <lanka.madan@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>
Tested-by: Uze Choi <uzchoi@samsung.com>
service/resource-manipulation/modules/common/primitiveResource/src/PresenceSubscriber.cpp

index 2b35cb6..a3284e7 100644 (file)
@@ -36,7 +36,7 @@ namespace OIC
                     const std::string&, OCConnectivityType, SubscribeCallback);
 
             invokeOCFunc(static_cast<SubscribePresence>(OC::OCPlatform::subscribePresence),
-                    handle, host, connectivityType, presenceHandler);
+                    handle, host, connectivityType, std::move(presenceHandler));
         }
 
         void subscribePresence(OCDoHandle& handle, const std::string& host,
@@ -47,7 +47,7 @@ namespace OIC
                     const std::string&, const std::string&, OCConnectivityType, SubscribeCallback);
 
             invokeOCFunc(static_cast<SubscribePresence>(OC::OCPlatform::subscribePresence),
-                    handle, host, resourceType, connectivityType, presenceHandler);
+                    handle, host, resourceType, connectivityType, std::move(presenceHandler));
         }
 
         void unsubscribePresence(OCDoHandle handle)
@@ -71,7 +71,7 @@ namespace OIC
                 OCConnectivityType connectivityType, SubscribeCallback presenceHandler) :
                 m_handle{ nullptr }
         {
-            subscribePresence(m_handle, host, connectivityType, presenceHandler);
+            subscribePresence(m_handle, host, connectivityType, std::move(presenceHandler));
         }
 
         PresenceSubscriber::PresenceSubscriber(const std::string& host,
@@ -79,7 +79,8 @@ namespace OIC
                 SubscribeCallback presenceHandler) :
                 m_handle{ nullptr }
         {
-            subscribePresence(m_handle, host, resourceType, connectivityType, presenceHandler);
+            subscribePresence(m_handle, host, resourceType, connectivityType,
+                    std::move(presenceHandler));
         }
 
         PresenceSubscriber::~PresenceSubscriber()