Prefer swap() member function to std::swap. We were hitting the

inefficient generic std::swap for some derived types.

1 files changed, 2 insertions, 3 deletions

diff --git a/builtin_set_color.cpp b/builtin_set_color.cpp
index 6e8f55b0..0fdc1ac0 100644
--- a/builtin_set_color.cpp
+++ b/builtin_set_color.cpp
@@ -235,9 +235,8 @@ static int builtin_set_color(parser_t &parser, wchar_t **argv)
     output_set_writer(saved_writer_func);
 
     /* Output the collected string */
-    std::string local_output;
-    std::swap(builtin_set_color_output, local_output);
-    stdout_buffer.append(str2wcstring(local_output));
+    stdout_buffer.append(str2wcstring(builtin_set_color_output));
+    builtin_set_color_output.clear();
 
     return STATUS_BUILTIN_OK;
 }