node-xml

node-xml copied to clipboard

fix markdown texts

bezzad

krkk

So that this package can be used outside of the Node.js environment e.g. React Native, I've removed dependencies on `util` (replaced with `console`) and `fs` (removed method taking a file).

adamski

Should not readFile an entire file.

1

fs.readFile, defeats the purpose for a SaxParser. This is loading the entire xml into memory and causes errors when loading large xml files. SaxParser.prototype.parseFile = function(filename) { //This function will...

uptownhr

There might be a better way to do this, but I was having issues with xml files with different encodings (utf-8) to us-ascii. This fixes it for me

timhc22

- Some xml file contains left and right quote and quote character in CDATA tag for ex: “a text inside left and right quote” - Left and right quote in...

gunivan

Is there a reason why   results in the empty text event? It should still be a space, isn't it? https://github.com/robrighter/node-xml/blob/master/lib/node-xml.js#L682

intenter

Mocha reports variable leaks into global scope

2

Mocha (http://visionmedia.github.com/mocha/) reports some errors in a projet of mine that uses SaxParser(). For instance: Error: global leak detected: util Error: global leaks detected: iRet, theatts, splits, nameobject, that

forissier

The api says it returns a boolean that says the xml is valid or not but it does not return anything.

mikeyjm145

I argue that `[]` should be used instead of `new Array()`, and especially instead of `new Array(arg0, arg1, ...)` to avoid confusion (i.e. I was confused when I saw it)....

coolaj86