RFR 8005280: (props) Improve test coverage for small XML parser
Alan Bateman
Alan.Bateman at oracle.com
Fri Dec 21 20:35:16 UTC 2012
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
More information about the core-libs-dev
mailing list