[jdom-interest] question about running speed - patch

Bradley S. Huffman hip at a.cs.okstate.edu
Thu Apr 11 18:33:21 PDT 2002


Jason Hunter writes:

> > PS I noticed that characters() checks inCDATA, but ignorableWhitespace()
> > doesn't. Is this correct? Should both characters() and ignorableWhitespace(
> > call the same internal method, to avoid duplicate code?
> 
> I expect you're right.

Looks like it could be changed to:

    if ( !ignoringWhite) {
        characters(ch, start, length);
    }

But right now Alex is right, it doesn't catch a transition between CDATA
sections and text. A bug.

Brad



More information about the jdom-interest mailing list