Merge pull request #492 from sjahu/pure-comment-regex-fix

Relax Pure::Parser's comment regex to allow any characters in a multi-line comment

Author: byroot

lib/json/pure/parser.rb

@@ -39,10 +39,7 @@ class Parser < StringScanner
          //[^\n\r]*[\n\r]| # line comments
          /\*               # c-style comments
          (?:
-          [^*/]|        # normal chars
-          /[^*]|        # slashes that do not start a nested comment
-          \*[^/]|       # asterisks that do not end this comment
-          /(?=\*/)      # single slash before this comment's end
+          [\s\S]*?         # any char, repeated lazily
          )*
            \*/               # the End of this comment
            |[ \t\r\n]+       # whitespaces: space, horizontal tab, lf, cr

test/json/json_parser_test.rb

@@ -272,6 +272,14 @@ def test_parse_comments
 EOT
     assert_raise(ParserError) { parse(json) }
     json = <<EOT
+{
+  "key1":"value1"  /* multi line
+                    // nested eol comment
+                    /* legal nested multi line comment start sequence */
+}
+EOT
+    assert_equal({ "key1" => "value1" }, parse(json))
+    json = <<EOT
 {
   "key1":"value1"  /* multi line
                    // nested eol comment