Close #33 Single-hyphens allowed in comments in strict mode

lib/sax.js

@@ -530,14 +530,19 @@ function write (chunk) {
           if (parser.comment) emitNode(parser, "oncomment", parser.comment)
           parser.comment = ""
         } else {
-          strictFail(parser, "Invalid comment")
           parser.comment += "-" + c
+          parser.state = S.COMMENT
         }
       continue
 
       case S.COMMENT_ENDED:
-        if (c !== ">") strictFail(parser, "Malformed comment")
-        else parser.state = S.TEXT
+        if (c !== ">") {
+          strictFail(parser, "Malformed comment")
+          // allow <!-- blah -- bloo --> in non-strict mode,
+          // which is a comment of " blah -- bloo "
+          parser.comment += "--" + c
+          parser.state = S.COMMENT
+        } else parser.state = S.TEXT
       continue
 
       case S.CDATA:

test/issue-30.js

@@ -0,0 +1,24 @@
+// https://github.com/isaacs/sax-js/issues/33
+require(__dirname).test
+  ( { xml : "<xml>\n"+
+            "<!-- \n"+
+            "  comment with a single dash- in it\n"+
+            "-->\n"+
+            "<data/>\n"+
+            "</xml>"
+
+    , expect :
+      [ [ "opentag", { name: "xml", attributes: {} } ]
+      , [ "text", "\n" ]
+      , [ "comment", " \n  comment with a single dash- in it\n" ]
+      , [ "text", "\n" ]
+      , [ "opentag", { name: "data", attributes: {} } ]
+      , [ "closetag", "data" ]
+      , [ "text", "\n" ]
+      , [ "closetag", "xml" ]
+      ]
+    , strict : true
+    , opt : {}
+    }
+  )
+