more clean up of bracket processing

CLIUtils · Jan 2, 2025 · 39fa3f4 · 39fa3f4
1 parent 7250801
commit 39fa3f4
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 10 deletions.
diff --git a/include/CLI/impl/Option_inl.hpp b/include/CLI/impl/Option_inl.hpp
@@ -693,10 +693,6 @@ CLI11_INLINE int Option::_add_result(std::string &&result, std::vector<std::stri
         result.erase(result.begin());
         bool skipSection{false};
         for(auto &var : CLI::detail::split_up(result, ',')) {
-            if(var == result) {
-                skipSection = true;
-                break;
-            }
             if(!var.empty()) {
                 result_count += _add_result(std::move(var), res);
             }

diff --git a/tests/OptionTypeTest.cpp b/tests/OptionTypeTest.cpp
@@ -1227,17 +1227,17 @@ TEST_CASE_METHOD(TApp, "vectorSingleArg", "[optiontype]") {
 
 TEST_CASE_METHOD(TApp, "vectorEmptyArg", "[optiontype]") {
 
-    std::vector<std::string> cv{"test"};
+    std::vector<std::string> cv{ "test" };
     app.add_option("-c", cv);
-    args = {"-c", "test1", "[]"};
+    args = { "-c","test1", "[]"};
 
     run();
-    CHECK(cv.size() == 1);
-    args = {"-c", "test1", "[[]]"};
+    CHECK(cv.size()==1);
+    args = { "-c","test1", "[[]]"};
 
     run();
-    CHECK(cv.size() == 2);
-    CHECK(cv[1] == "[]");
+    CHECK(cv.size()==2);
+    CHECK(cv[1]=="[]");
 }
 
 TEST_CASE_METHOD(TApp, "vectorDoubleArg", "[optiontype]") {
@@ -1264,6 +1264,18 @@ TEST_CASE_METHOD(TApp, "vectorEmpty", "[optiontype]") {
     CHECK(cv.empty());
 }
 
+TEST_CASE_METHOD(TApp, "vectorVectorArg", "[optiontype]") {
+
+    std::vector<std::vector<std::string>> cv{};
+    app.add_option("-c", cv);
+    args = { "-c","[[a,b]]"};
+
+    run();
+    CHECK(cv.size()==1);
+    CHECK(cv[0].size() == 2);
+    CHECK(cv[0][0]=="a");
+}
+
 TEST_CASE_METHOD(TApp, "OnParseCall", "[optiontype]") {
 
     int cnt{0};