Change TODO on trailing comma to 'someday'

8a9a8ab2 · Kamal Marhubi · c0d00748 · 8a9a8ab2
Commit 8a9a8ab2 authored Nov 09, 2015 by Kamal Marhubi
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

json.c++ c++/src/capnp/compat/json.c++ +2 -2

No files found.
--- a/c++/src/capnp/compat/json.c++
+++ b/c++/src/capnp/compat/json.c++
@@ -422,7 +422,7 @@ public:
      values.add(kj::mv(orphan));

      if (consumeWhitespace(), nextChar() != ']') {
-        // TODO(soon): This incorrectly allows a trailing comma.
+        // TODO(someday): The JSON spec forbids a trailing comma, but we allow it.
        consume(',');
      }
    }
@@ -458,7 +458,7 @@ public:
      fields.add(kj::mv(orphan));

      if (consumeWhitespace(), nextChar() != '}') {
-        // TODO(soon): This incorrectly allows a trailing comma.
+        // TODO(someday): The JSON spec forbids a trailing comma, but we allow it.
        consume(',');
      }
    }