Fix for comments

autodesk-forks · Dec 13, 2024 · 51d70ea · 51d70ea
1 parent b168c31
commit 51d70ea
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/OpenColorIO/ParseUtils.cpp b/src/OpenColorIO/ParseUtils.cpp
@@ -691,7 +691,7 @@ bool StrEqualsCaseIgnore(const std::string & a, const std::string & b)
 }
 
 // Find the end of a name from a list contained in a string.
-// The element of the list are separeted by ":" or ",".
+// The element of the list are separeted by a character defined by the sep parameter.
 // The name can be surrounded by quotes to enable name including theses symbols.
 static int findEndOfName(const std::string & s, size_t start, char sep)
 {

diff --git a/tests/cpu/ParseUtils_tests.cpp b/tests/cpu/ParseUtils_tests.cpp
@@ -449,6 +449,9 @@ OCIO_ADD_TEST(ParseUtils, split_string_env_style)
     OCIO_CHECK_EQUAL("a", outputvec[1]);
     OCIO_CHECK_EQUAL("test", outputvec[2]);
 
+    // If the string contains a comma, 
+    // it is chosen as the separator character rather than the colon 
+    // (even if it is within quotes and therefore not used as such).
     outputvec = OCIO::SplitStringEnvStyle("   \"This  , is   \":   a:   test  ");
     OCIO_CHECK_EQUAL(1, outputvec.size());
     OCIO_CHECK_EQUAL("\"This  , is   \":   a:   test", outputvec[0]);