diff --git a/src/common/rclconfig.cpp b/src/common/rclconfig.cpp index 556604e3..a2a6d4fd 100644 --- a/src/common/rclconfig.cpp +++ b/src/common/rclconfig.cpp @@ -516,13 +516,10 @@ string RclConfig::getMimeHandlerDef(const std::string &mtype, bool filtertypes) return hs; } -/** - * @param whole is like "base value ; attr1 = somthing; attr2 = somethelse" - * There is no way to escape a semi-colon inside whole - */ bool RclConfig::valueSplitAttributes(const string& whole, string& value, - ConfSimple& attrs) + ConfSimple& attrs) { + /* There is currently no way to escape a semi-colon */ string::size_type semicol0 = whole.find_first_of(";"); value = whole.substr(0, semicol0); string attrstr; diff --git a/src/common/rclconfig.h b/src/common/rclconfig.h index 4f81e794..07b2ede8 100644 --- a/src/common/rclconfig.h +++ b/src/common/rclconfig.h @@ -171,8 +171,11 @@ class RclConfig { /** mimeconf: get input filter for mimetype */ string getMimeHandlerDef(const string &mimetype, bool filtertypes=false); - /** For lines like: "name = some value; and additional; attributes" - * Split the value and store the attributes in a ConfSimple */ + + /** For lines like: "name = some value; attr1 = value1; attr2 = val2" + * Separate the value and store the attributes in a ConfSimple + * @param whole the raw value. No way to escape a semi-colon in there. + */ bool valueSplitAttributes(const string& whole, string& value, ConfSimple& attrs);