Hi Dave!
From: "David White" <[EMAIL PROTECTED]>
> Is there a problem with the mailing list???
>
> I just tried to send the following message to it a couple of times and it
> bounced back to me. Maybe it's me, I don't know...
I've cc'd the list just to check. Maybe its the attachments on your original
mail or the list server's down, I'm not sure.
This looks great, thank you! By all means check this straight into CVS if
you like. A couple of comments below..
> > -----Original Message-----
> > From: David White
> > Sent: Friday, May 18, 2001 8:41 AM
> > To: '[EMAIL PROTECTED]'
> > Subject: Code for parsing large documents
> >
> > Hey James,
> >
> > I don't know if you've been working on this yet, but I've implemented
the
> > functionality we talked about to enhance the stream based "pruning mode"
> > stuff. (I know I said for you to go ahead an do it, but I was just
> > itching
> > to
> > try it out...)
> >
> > Here's a diff file (starting from the src/org/dom4j directory) with the
> > changes
> > to the existing classes.
> >
> > <<dom4j.diff>>
> > The new classes/interfaces are attached here:
> >
> > <<ElementPath.java>> <<DispatchHandler.java>>
> > Let me know what you think of this stuff. I think it covers what we
> > discussed, but
> > I could be wrong (and often am :^).
me too! :-)
> > My biggest question at this point is what are your thoughts on breaking
> > the
> > old
> > ElementHandler interface? Should we try to rename some of this stuff to
> > try
> > to be
> > backward compatible? It's kinda tricky, as the endElement method of
> > SAXContentHandler needs to follow one interface or the other. I didn't
> > remove the
> > handle(Element element) method of the ElementHandler interface yet, as I
> > wanted
> > to hear from you on this. Let me know what you think.
I think now is a good time to make the change to the ElementHandler. Its
such a small, simple interface I hope it won't affect too many people too
much. (Someone shout out now if its a problem!)
Shall we go for the following?
public interface ElementHandler {
public void onStart( ElementPath path );
public void onEnd( ElementPath path );
}
> > If you think this looks ok, then I can check this stuff in. Otherwise,
> > just
> > let me know
> > if you have changes or something else in mind.
Do you want to go ahead and check your changes in? (Have you managed to do a
non-anonymous CVS checkout OK?). I'll do it if you prefer but otherwise, go
ahead.
BTW remember to put your name as the author in the javadoc comment & add the
licence on the top.
> > Thanks,
> >
> > Dave
Thank you Dave!
James
_________________________________________________________
Do You Yahoo!?
Get your free @yahoo.com address at http://mail.yahoo.com
_______________________________________________
dom4j-dev mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/dom4j-dev
