options: Fix a small issue with vector formatting

f3d-app · Dec 22, 2024 · 8f3c4a3 · 8f3c4a3
1 parent 25afbaa
commit 8f3c4a3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/library/private/options_tools.h.in b/library/private/options_tools.h.in
@@ -264,7 +264,7 @@ std::string format(const std::vector<double>& var)
   unsigned int i = 0;
   for (auto& elem : var)
   {
-    stream << ((i > 0) ? ", " : "") << options_tools::format(elem);
+    stream << ((i > 0) ? "," : "") << options_tools::format(elem);
     i++;
   }
   return stream.str();

diff --git a/library/testing/TestSDKOptions.cxx b/library/testing/TestSDKOptions.cxx
@@ -73,11 +73,14 @@ int TestSDKOptions(int argc, char* argv[])
   test("set/get string", std::get<std::string>(opt.get("model.color.texture")) == "test");
 
   // Test double vector
-  opt.setAsString("render.background.color", "0.1, 0.2, 0.4");
-  test("setAsString vector<double>", opt.getAsString("render.background.color") == "0.1, 0.2, 0.4");
+  opt.setAsString("render.background.color", "0.1,0.2,0.4");
+  test("setAsString vector<double>", opt.getAsString("render.background.color") == "0.1,0.2,0.4");
+
+  opt.setAsString("render.background.color", "0.1, 0.3, 0.4");
+  test("setAsString spaces vector<double>", opt.getAsString("render.background.color") == "0.1,0.3,0.4");
 
   opt.render.background.color = { 0.1, 0.2, 0.5 };
-  test("getAsString vector<double>", opt.getAsString("render.background.color") == "0.1, 0.2, 0.5");
+  test("getAsString vector<double>", opt.getAsString("render.background.color") == "0.1,0.2,0.5");
 
   opt.set("render.background.color", std::vector<double>{ 0.1, 0.2, 0.3 });
   test("set/get vector<double>",