← Back to team overview

nrtb-core team mailing list archive

[Branch ~fpstovall/nrtb/fps-sprint-003] Rev 26: Added move sematics to nrtb::abs_queue. Fix for bug 1217388; unit tested, no regressions apparent.

 

------------------------------------------------------------
revno: 26
committer: Rick Stovall <fpstovall>
branch nick: ricks-sprint-003
timestamp: Tue 2013-08-27 11:22:39 -0400
message:
  Added move sematics to nrtb::abs_queue. Fix for bug 1217388; unit tested, no regressions apparent.
modified:
  cpp/common/abs_queue/abs_queue.h


--
lp:~fpstovall/nrtb/fps-sprint-003
https://code.launchpad.net/~fpstovall/nrtb/fps-sprint-003

Your team NRTB Core is subscribed to branch lp:~fpstovall/nrtb/fps-sprint-003.
To unsubscribe from this branch go to https://code.launchpad.net/~fpstovall/nrtb/fps-sprint-003/+edit-subscription
=== modified file 'cpp/common/abs_queue/abs_queue.h'
--- cpp/common/abs_queue/abs_queue.h	2013-06-30 18:37:44 +0000
+++ cpp/common/abs_queue/abs_queue.h	2013-08-27 15:22:39 +0000
@@ -117,7 +117,7 @@
     in_count++;
     {
       std::unique_lock<std::mutex> lock(mylock);
-      buffer.push(item);
+      buffer.push(std::move(item));
     }
     signal.notify_one();
   }
@@ -139,7 +139,7 @@
     T returnme = buffer.front();
     buffer.pop();
     out_count++;
-    return returnme;
+    return std::move(returnme);
   }
   else
   {