SkPDF: fix pessimizing-move in PDFA code
authorhalcanary <halcanary@google.com>
Fri, 8 Apr 2016 16:55:20 +0000 (09:55 -0700)
committerCommit bot <commit-bot@chromium.org>
Fri, 8 Apr 2016 16:55:21 +0000 (09:55 -0700)
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1863393004

Review URL: https://codereview.chromium.org/1863393004

src/pdf/SkPDFMetadata.cpp

index 4ca1903d21b2a23fce8268b994b513044a30c7ec..a5476a2df0980e6755be98f62f3038be15c2f416 100644 (file)
@@ -122,7 +122,7 @@ static SkString sk_string_printf(const char* format, ...) {
     va_end(args);
     SkASSERT(check == length);
     SkASSERT(string[length] == '\0');
-    return std::move(string);
+    return string;
 #else  // C99/C++11 standard vsnprintf
     // TODO: When all compilers support this, remove windows-specific code.
     va_list args;
@@ -143,7 +143,7 @@ static SkString sk_string_printf(const char* format, ...) {
     va_end(args);
     SkASSERT(check == length);
     SkASSERT(string[length] == '\0');
-    return std::move(string);
+    return string;
 #endif
 }