LGTM

http://codereview.chromium.org/548198/diff/1/3
File tools/csvparser.js (right):

http://codereview.chromium.org/548198/diff/1/3#newcode72
tools/csvparser.js:72: fields.push(field.replace(doubleQuoteRe, '"'));
Is this covered by any tests?

http://codereview.chromium.org/548198

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to