Apply suggestions from code review

Co-authored-by: Alon Zakai <[email protected]>

Author: caiiiycuk

src/support/string.h

@@ -32,7 +32,11 @@ namespace wasm::String {
 // Creates a vector of the split parts of a string, by a delimiter.
 class Split : public std::vector<std::string> {
 private:
+  // If we split on newlines then we do not need to handle bracketing at all. Otherwise,
+  // splitting on say "," does require us to understanding the scoping of brackets, e.g.,
+  // "foo(x, y),bar" should be split as "foo(x, y)", "bar".
   bool needToHandleBracketingOperations = true;
+
   void split(const std::string& input, const std::string& delim) {
     size_t lastEnd = 0;
     while (lastEnd < input.size()) {
@@ -47,7 +51,10 @@ class Split : public std::vector<std::string> {
   }
   friend String::Split handleBracketingOperators(String::Split split);
 
+
 public:
+  // This can be used when we want to split on newlines if there are any, and if there are not,
+  // then using the given delimiter.
   struct NewLineOr {
     const std::string delim;
     explicit NewLineOr(const std::string& delim) : delim(delim) {}