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

Joe Wang huizhe.wang at oracle.com
Fri Dec 21 21:18:15 UTC 2012



On 12/21/2012 12:35 PM, Alan Bateman wrote:
> 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).

I still have the source=1.5 mentality :)  In 2009, we dropped JDK 1.4 
support for the jaxp standalone. I guess it's time to make a jump to 
source=1.7 now!

I re-generated the webrev just to keep the record.  I will push after a 
new jprt test.

-Joe

>
> -Alan



More information about the core-libs-dev mailing list