RFR (JAXP) 8029955 : AIOB in XMLEntityScanner.scanLiteral upon parsing literals with > 100 LF chars
Daniel Fuchs
daniel.fuchs at oracle.com
Tue Dec 17 12:10:14 UTC 2013
Hi Joe,
The fix looks good - though I wonder at whether incrementing
whiteSpaceLookup by a fix amount wouldn't be better than
doubling its length.
best regards,
-- daniel
On 12/16/13 8:31 PM, huizhe wang wrote:
> Hi,
>
> This is a quick fix for a whitespace buffer that was not adjusted
> properly in one of the two cases. The buffer, whiteSpaceLookup, is
> filled in two cases and adjusted properly the 2nd time. The code is
> moved into a method storeWhiteSpace so that it's shared for the 1st case
> as well.
>
> Note at line 1175, there is no need to save character 0x20 since all
> whitespace characters will later be replaced with character 0x20.
>
> webrevs:
> http://cr.openjdk.java.net/~joehw/jdk8/8029955/webrev/
>
> Thanks,
> Joe
More information about the core-libs-dev
mailing list