RFR 8005280: (props) Improve test coverage for small XML parser (original) (raw)

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