[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