[jdom-interest] question about running speed - patch

[Jason Hunter]

> - The beta 8 version is fine, because it uses StringBuffer.substring(),
> which creates a String that doesn't share the underlying char array. But,
> it's not JDK 1.1 compatible.

Actually, we made the JDK 1.1 change between b8-rc1 and b8 final, so I
expect the poor performance should be in the b8 release.  That's
consistent with Dennis saying build performance was noticeably down in
b8 from b7.

> 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.

>   ------------------------------------------------------------------------
>                          Name: SAXHandlerDiff.txt
>    SAXHandlerDiff.txt    Type: Plain Text (text/plain)
>                      Encoding: quoted-printable
> 
>                       Name: TextBuffer.java
>    TextBuffer.java    Type: Plain Text (text/plain)
>                   Encoding: quoted-printable

Does TextBuffer have to be public?

-jh-