8004371: (props) Properties.loadFromXML needs small footprint XML parser as fallback when JAXP is not present (original) (raw)
Mandy Chung mandy.chung at oracle.com
Wed Dec 19 00:27:45 UTC 2012
- Previous message: 8004371: (props) Properties.loadFromXML needs small footprint XML parser as fallback when JAXP is not present
- Next message: RFR: 8003246: Add Supplier to ThreadLocal
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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
- Previous message: 8004371: (props) Properties.loadFromXML needs small footprint XML parser as fallback when JAXP is not present
- Next message: RFR: 8003246: Add Supplier to ThreadLocal
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]