[webkit-reviews] review denied: [Bug 41427] Add expat based XMLDocumentParser : [Attachment 60147] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Jul 27 07:34:03 PDT 2010
Adam Barth <abarth at webkit.org> has denied Patrick R. Gansterer
<paroga at paroga.com>'s request for review:
Bug 41427: Add expat based XMLDocumentParser
https://bugs.webkit.org/show_bug.cgi?id=41427
Attachment 60147: Patch
https://bugs.webkit.org/attachment.cgi?id=60147&action=review
------- Additional Comments from Adam Barth <abarth at webkit.org>
I really don't like the idea of adding a third (!) XML parser. It's bad enough
that we have three. If we can improve perf by switching our XML parser, that's
great, but we should kill the old one. IMHO, adding the second XML parser was
a mistake too.
More information about the webkit-reviews
mailing list