8004371: (props) Properties.loadFromXML needs small footprint XML parser as fallback when JAXP is not present
Mandy Chung
mandy.chung at oracle.com
Wed Dec 19 00:27:45 UTC 2012
On 12/18/12 6:17 AM, Alan Bateman wrote:
>
> I've refreshed the webrev here to take in another change from Joe to
> the root element handling in PropertiesDefaultHandler.
>
> http://cr.openjdk.java.net/~alanb/8004371/webrev.03/
>
> I'd like to get this into jdk8/tl in the next few days if possible.
>
> On tests on then I've expanded LoadAndStoreXML to include a directory
> of invalid XML documents (those that should cause
> InvalidProperitesFormatException to be thrown) so that improves the
> test coverage a bit. Joe has several other tests in the works so I've
> suggested to him that he does a follow-up change in the coming weeks
> with those tests.
>
Looks okay to me. It's good that you have expanded the test to improve
the coverage testing invalid XML documents. Looking forward to the
additional tests Joe will be adding in the coming weeks.
Mandy
More information about the core-libs-dev
mailing list