Avoid std::move() for copy elision

This could disable zero-copy optimisations, which is the opposite of
what we want.

Utilities/StrFmt.cpp

@@ -402,7 +402,7 @@ std::vector<std::string> fmt::split(const std::string& source, std::initializer_
 		result.push_back(source.substr(cursor_begin));
 	}
 
-	return std::move(result);
+	return result;
 }
 
 std::string fmt::trim(const std::string& source, const std::string& values)