Cdata about node-expat HOT 5 CLOSED

Guys, dunno if it's correct, but seem to work for me.
https://gist.github.com/725268

I'm not sure how to push it, so tell me if it's correct patch I'll figure it out.

from node-expat.

So far, expat yields cdata as text. Isn't that enough? How do you want to handle cdata other than text?

Your proposed patch adds just the events, while cdata is still emitted as text. How are you going to deal with that? I just want to understand before I merge the code.

Also, how would you like to be mentioned in the contributors field of package.json?

from node-expat.

In my saxparser wrapper I use flag isInCdata using new methods for setting/unsetting it and then check it on 'text' event to fire my internal onCharacters or onCdata.

Pleast check it carefuly, it's my first cpp code ever.
I'm not pretending for it, but won't act against it either :)

from node-expat.

Thanks, it's included in node-expat 1.2.0.

Please notice that I renamed the events from startCdataElement/endCdataElement to startCdata/endCdata because CData sections are no XML elements and it's shorter.

from node-expat.

They're so called in expat itself, well doesn't matter really. Thanks a lot!

from node-expat.