Title: [248208] releases/WebKitGTK/webkit-2.24/Source/WTF
Revision
248208
Author
[email protected]
Date
2019-08-03 20:22:14 -0700 (Sat, 03 Aug 2019)

Log Message

Merge r243115 - [WTF] Remove redundant std::move in StringConcatenate
https://bugs.webkit.org/show_bug.cgi?id=195798

Patch by Xan Lopez <[email protected]> on 2019-03-18
Reviewed by Darin Adler.

Remove redundant calls to WTFMove in return values for this
method. C++ will already do an implicit move here since we are
returning a local value where copy/move elision is not applicable.

* wtf/text/StringConcatenate.h:
(WTF::tryMakeStringFromAdapters):

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.24/Source/WTF/ChangeLog (248207 => 248208)


--- releases/WebKitGTK/webkit-2.24/Source/WTF/ChangeLog	2019-08-04 03:22:12 UTC (rev 248207)
+++ releases/WebKitGTK/webkit-2.24/Source/WTF/ChangeLog	2019-08-04 03:22:14 UTC (rev 248208)
@@ -1,3 +1,17 @@
+2019-03-18  Xan Lopez  <[email protected]>
+
+        [WTF] Remove redundant std::move in StringConcatenate
+        https://bugs.webkit.org/show_bug.cgi?id=195798
+
+        Reviewed by Darin Adler.
+
+        Remove redundant calls to WTFMove in return values for this
+        method. C++ will already do an implicit move here since we are
+        returning a local value where copy/move elision is not applicable.
+
+        * wtf/text/StringConcatenate.h:
+        (WTF::tryMakeStringFromAdapters):
+
 2019-03-19  Michael Catanzaro  <[email protected]>
 
         Build cleanly with GCC 9

Modified: releases/WebKitGTK/webkit-2.24/Source/WTF/wtf/text/StringConcatenate.h (248207 => 248208)


--- releases/WebKitGTK/webkit-2.24/Source/WTF/wtf/text/StringConcatenate.h	2019-08-04 03:22:12 UTC (rev 248207)
+++ releases/WebKitGTK/webkit-2.24/Source/WTF/wtf/text/StringConcatenate.h	2019-08-04 03:22:14 UTC (rev 248208)
@@ -278,7 +278,7 @@
 
         makeStringAccumulator(buffer, adapter, adapters...);
 
-        return WTFMove(resultImpl);
+        return resultImpl;
     }
 
     UChar* buffer;
@@ -288,7 +288,7 @@
 
     makeStringAccumulator(buffer, adapter, adapters...);
 
-    return WTFMove(resultImpl);
+    return resultImpl;
 }
 
 template<typename... StringTypes>
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to