fix cdata issue #132 (#133)

lib/parsers/ltx.js

@@ -14,6 +14,7 @@ var STATE_ATTR_EQ = 6
 var STATE_ATTR_QUOT = 7
 var STATE_ATTR_VALUE = 8
 var STATE_CDATA = 9
+var STATE_IGNORE_CDATA = 10
 
 var SaxLtx = module.exports = function SaxLtx () {
   EventEmitter.call(this)
@@ -83,6 +84,13 @@ var SaxLtx = module.exports = function SaxLtx () {
         if (endcomment !== -1) {
           pos = endcomment + 2 // target the '>' character
         }
+      } else if (state === STATE_IGNORE_CDATA) {
+        // if we're looping through a CDATA, fast-forward using
+        // indexOf to the first end-CDATA character ]]>
+        var endCDATA = data.indexOf(']]>', pos)
+        if (endCDATA !== -1) {
+          pos = endCDATA + 2 // target the '>' character
+        }
       }
 
       var c = data.charCodeAt(pos)
@@ -104,7 +112,7 @@ var SaxLtx = module.exports = function SaxLtx () {
             if (cData) {
               this.emit('text', cData)
             }
-            state = STATE_IGNORE_COMMENT
+            state = STATE_TEXT
           }
           break
         case STATE_TAG_NAME:

test/cdata-test.js

@@ -4,6 +4,7 @@ var vows = require('vows')
 var assert = require('assert')
 var ltx = require('..')
 var parsers = require('../lib/parsers')
+var h = ltx.createElement
 
 var Parser = parsers.find(function (parser) {
   return (parser.name === 'SaxLtx')
@@ -24,6 +25,10 @@ vows.describe('sax_ltx').addBatch({
       var el = parse('<root><![CDATA[Content &amp; "more content&quot;]]></root>')
       assert.strictEqual(el.name, 'root')
       assert.strictEqual(el.getText(), 'Content &amp; "more content&quot;')
+    },
+    'issue-132': () => {
+      var el = parse('<a><b><![CDATA[]]></b><b><![CDATA[--><c>&d;]]></b></a>')
+      assert.deepStrictEqual(el, h('a', null, h('b', null, ''), h('b', null, '--><c>&d;')))
     }
   }
 }).export(module)