Add Comment start state errors.

Author: inikulin

lib/common/error_codes.js

@@ -15,5 +15,6 @@ module.exports = {
     cdataInHtmlContent: 'cdata-in-html-content',
     malformedComment: 'malformed-comment',
     eofInScriptHtmlComment: 'eof-in-script-html-comment',
-    nestedComment: 'nested-comment'
+    nestedComment: 'nested-comment',
+    abruptComment: 'abrupt-comment'
 };

lib/tokenizer/index.js

@@ -1577,25 +1577,14 @@ _[COMMENT_START_STATE] = function commentStartState(cp) {
     if (cp === $.HYPHEN_MINUS)
         this.state = COMMENT_START_DASH_STATE;
 
-    else if (cp === $.NULL) {
-        this.currentToken.data += UNICODE.REPLACEMENT_CHARACTER;
-        this.state = COMMENT_STATE;
-    }
-
     else if (cp === $.GREATER_THAN_SIGN) {
+        this._err(ERR.abruptComment);
         this.state = DATA_STATE;
         this._emitCurrentToken();
     }
 
-    else if (cp === $.EOF) {
-        this._emitCurrentToken();
-        this._emitEOFToken();
-    }
-
-    else {
-        this.currentToken.data += toChar(cp);
-        this.state = COMMENT_STATE;
-    }
+    else
+        this._reconsumeInState(COMMENT_STATE);
 };
 
 
@@ -1605,12 +1594,6 @@ _[COMMENT_START_DASH_STATE] = function commentStartDashState(cp) {
     if (cp === $.HYPHEN_MINUS)
         this.state = COMMENT_END_STATE;
 
-    else if (cp === $.NULL) {
-        this.currentToken.data += '-';
-        this.currentToken.data += UNICODE.REPLACEMENT_CHARACTER;
-        this.state = COMMENT_STATE;
-    }
-
     else if (cp === $.GREATER_THAN_SIGN) {
         this.state = DATA_STATE;
         this._emitCurrentToken();
@@ -1623,8 +1606,7 @@ _[COMMENT_START_DASH_STATE] = function commentStartDashState(cp) {
 
     else {
         this.currentToken.data += '-';
-        this.currentToken.data += toChar(cp);
-        this.state = COMMENT_STATE;
+        this._reconsumeInState(COMMENT_STATE);
     }
 };
 
@@ -1707,21 +1689,14 @@ _[COMMENT_END_DASH_STATE] = function commentEndDashState(cp) {
     if (cp === $.HYPHEN_MINUS)
         this.state = COMMENT_END_STATE;
 
-    else if (cp === $.NULL) {
-        this.currentToken.data += '-';
-        this.currentToken.data += UNICODE.REPLACEMENT_CHARACTER;
-        this.state = COMMENT_STATE;
-    }
-
     else if (cp === $.EOF) {
         this._emitCurrentToken();
         this._emitEOFToken();
     }
 
     else {
         this.currentToken.data += '-';
-        this.currentToken.data += toChar(cp);
-        this.state = COMMENT_STATE;
+        this._reconsumeInState(COMMENT_STATE);
     }
 };
 
@@ -1740,21 +1715,14 @@ _[COMMENT_END_STATE] = function commentEndState(cp) {
     else if (cp === $.HYPHEN_MINUS)
         this.currentToken.data += '-';
 
-    else if (cp === $.NULL) {
-        this.currentToken.data += '--';
-        this.currentToken.data += UNICODE.REPLACEMENT_CHARACTER;
-        this.state = COMMENT_STATE;
-    }
-
     else if (cp === $.EOF) {
         this._emitCurrentToken();
         this._emitEOFToken();
     }
 
     else {
         this.currentToken.data += '--';
-        this.currentToken.data += toChar(cp);
-        this.state = COMMENT_STATE;
+        this._reconsumeInState(COMMENT_STATE);
     }
 };
 
@@ -1772,21 +1740,14 @@ _[COMMENT_END_BANG_STATE] = function commentEndBangState(cp) {
         this._emitCurrentToken();
     }
 
-    else if (cp === $.NULL) {
-        this.currentToken.data += '--!';
-        this.currentToken.data += UNICODE.REPLACEMENT_CHARACTER;
-        this.state = COMMENT_STATE;
-    }
-
     else if (cp === $.EOF) {
         this._emitCurrentToken();
         this._emitEOFToken();
     }
 
     else {
         this.currentToken.data += '--!';
-        this.currentToken.data += toChar(cp);
-        this.state = COMMENT_STATE;
+        this._reconsumeInState(COMMENT_STATE);
     }
 };