Thanks Lance. Changed the for loop to a while loop:
http://cr.openjdk.java.net/~joehw/jdk12/8213734/webrev_02/
previous:
http://cr.openjdk.java.net/~joehw/jdk12/8213734/webrev_01/
-Joe
On 11/29/18, 11:17 AM, Joe Wang wrote:
Hi,
Please review a fix for the issue as reported that the SAXParser does
not close the underlying reader. This code always existed in Xerces,
but it was somehow removed as the comment for the closeReaders method
showed. The comment stated "readers are closed in the endEntity
method", that is true in general. However when the parsing is
interrupted, as in this case by an Exception, the endEntity method
won't get called, but closeReaders does through the Parser's cleanup
process. This patch recovers the original Xerces' code.
JBS: https://bugs.openjdk.java.net/browse/JDK-8213734
webrev: http://cr.openjdk.java.net/~joehw/jdk12/8213734/webrev/
Thanks,
Joe
