nrtb-core team mailing list archive
-
nrtb-core team
-
Mailing list archive
-
Message #00519
[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
{