On 18/10/2013 12:15, Daniel Fuchs wrote:

Yes - that would be simpler. Here is the new webrev:

http://cr.openjdk.java.net/~dfuchs/webrev_8016344/webrev.01/

-- daniel
I agree to, this is so much simpler.

Just on the name, TestStoreToXML makes it sound like a general test for storeToXML but we already have LoadAndStoreXML for that. I just wonder if the new test should be renamed to something that makes it clearer what it is testing. Also on the location, maybe we should just move all the tests for loadFromXML and storeToXML into a xml or some sub-directory to make it more obvious.

-Alan

Reply via email to