Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(35)

Unified Diff: webrtc/api/rtcerror.h

Issue 2675173003: Adding "adapter" ORTC objects on top of ChannelManager/BaseChannel/etc. (Closed)
Patch Set: Add memcheck suppression for end-to-end tests. Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/api/proxy.h ('k') | webrtc/api/rtpparameters.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/api/rtcerror.h
diff --git a/webrtc/api/rtcerror.h b/webrtc/api/rtcerror.h
index 1c130c0d6e824915029389a38b97aa802bea2c77..2ba78371eb0034f17b19833be32dc9fddf6fdc59 100644
--- a/webrtc/api/rtcerror.h
+++ b/webrtc/api/rtcerror.h
@@ -224,7 +224,7 @@ class RTCErrorOr {
// NOTE: Not explicit - we want to use RTCErrorOr<T> as a return type
// so it is convenient and sensible to be able to do 'return T()'
// when the return type is RTCErrorOr<T>.
- RTCErrorOr(T value) : value_(std::move(value)) {}
+ RTCErrorOr(T&& value) : value_(std::move(value)) {}
// Delete the copy constructor and assignment operator; there aren't any use
// cases where you should need to copy an RTCErrorOr, as opposed to moving
« no previous file with comments | « webrtc/api/proxy.h ('k') | webrtc/api/rtpparameters.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698