Bug 14791

Summary: Separate Qt and libxml based implementations of XMLTokenizer
Product: WebKit Reporter: Lars Knoll <lars.knoll>
Component: XMLAssignee: George Staikos <staikos>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: darin, hausmann
Priority: P2 Keywords: Qt
Version: 523.x (Safari 3)   
Hardware: Other   
OS: Other   
Attachments:
Description Flags
Patch to separate both XMLTokenizer implementations. eric: review-

Description Lars Knoll 2007-07-27 14:06:14 PDT
The attached patch separates the Qt and libxml based implementations of
XMLTokenizer into separate files while keeping the common code in XMLTokenizer.cpp
Comment 1 Lars Knoll 2007-07-27 14:08:13 PDT
Created attachment 15708 [details]
Patch to separate both XMLTokenizer implementations.
Comment 2 Darin Adler 2007-08-11 15:43:13 PDT
Sorry I haven't reviewed this yet. I know I'm the one who asked for it! I'll get to it ASAP (unless someone else reviews it first).
Comment 3 Lars Knoll 2007-08-12 00:42:38 PDT
No problem. The patch does not fix the fix the build system for the Safari builds btw (as I didn't have a Mac at hand). I was hoping you could add these pieces :)
Comment 4 Eric Seidel (no email) 2007-09-29 12:51:19 PDT
FYI, this patch will conflict with the one attached to bug 15302 (mathml bug, also up for review)
Comment 5 Eric Seidel (no email) 2007-09-30 12:08:26 PDT
Lars, I think we should just chat about this on irc.  I've done lots of these types of splits -- they're easy to do, but they tend to go stale as patches *very quickly*.  If you grab me (or any other reviewer) on irc, it would be real quick to just re-do this patch from scratch and get it reviewed immediately.  I'd be happy to assist in that if I'm around.
Comment 6 Eric Seidel (no email) 2007-10-03 09:48:35 PDT
Comment on attachment 15708 [details]
Patch to separate both XMLTokenizer implementations.

This should be re-done and reviewed immediately.  I'm certain this patch has rotten by now. :(
Comment 7 Simon Hausmann 2009-11-21 01:27:00 PST

*** This bug has been marked as a duplicate of bug 20437 ***