Fixed unfair comparison; std::move must be used in c++ push_back(). STC has almost identical performance to c++ STL, but STC cstr has 23 chars short strings vs typically 15 for c++ std::string, so STC is faster in this range.

1 files changed, 2 insertions, 2 deletions

diff --git a/benchmarks/misc/sso_bench.cpp b/benchmarks/misc/sso_bench.cpp
index 08444320..5ff54be9 100644
--- a/benchmarks/misc/sso_bench.cpp
+++ b/benchmarks/misc/sso_bench.cpp
@@ -32,7 +32,7 @@ static void sromutrio(uint64_t seed) {
 static const char CHARS[] = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz=+-";
 
 static const int BENCHMARK_SIZE = 5000000;
-static const int MAX_STRING_LENGTH = 25;
+static const int MAX_STRING_LENGTH = 30;
 
 using time_point = std::chrono::high_resolution_clock::time_point;
 
@@ -43,7 +43,7 @@ void addRandomString_STD(std::vector<std::string>& vec, const int length) {
         p[i] = CHARS[romutrio() & 63];
     }
     s.append(s);
-    vec.push_back(s);
+    vec.push_back(std::move(s));
 }
 
 void addRandomString_STC(svec& vec, const int length) {