On 21/12/2012 19:56, Joe Wang wrote:
:
I searched by properties.dtd and found the CDATA usage. So I went
ahead experimenting what would not be rejected by the regular
provider and tested against the small parser. I agree the cases other
than CDATA are rare.
Okay.
In CompatibilityTest.loadPropertyFile then it looks like the file is
not closed.
Properties' loadFromXML stated that it will close the stream after
loading. But it doesn't hurt to double-check.
New webrev:
http://cr.openjdk.java.net/~joehw/jdk8/8005280/webrev/
What you have now is okay but I think it would be nicer if it were
changed to use try-with-resources (no need to generate a new webrev for
that).
-Alan
