[WTF] Remove redundant std::move in StringConcatenate
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Mar 2019 23:51:42 +0000 (23:51 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Mar 2019 23:51:42 +0000 (23:51 +0000)
https://bugs.webkit.org/show_bug.cgi?id=195798

Patch by Xan Lopez <xan@igalia.com> 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):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@243115 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WTF/ChangeLog
Source/WTF/wtf/text/StringConcatenate.h

index 3ecb228..af2c29f 100644 (file)
@@ -1,3 +1,17 @@
+2019-03-18  Xan Lopez  <xan@igalia.com>
+
+        [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-16  Darin Adler  <darin@apple.com>
 
         Improve normalization code, including moving from unorm.h to unorm2.h
index 1c501ed..c5d0201 100644 (file)
@@ -278,7 +278,7 @@ String tryMakeStringFromAdapters(StringTypeAdapter adapter, StringTypeAdapters .
 
         makeStringAccumulator(buffer, adapter, adapters...);
 
-        return WTFMove(resultImpl);
+        return resultImpl;
     }
 
     UChar* buffer;
@@ -288,7 +288,7 @@ String tryMakeStringFromAdapters(StringTypeAdapter adapter, StringTypeAdapters .
 
     makeStringAccumulator(buffer, adapter, adapters...);
 
-    return WTFMove(resultImpl);
+    return resultImpl;
 }
 
 template<typename... StringTypes>