RFR: 8016344 - (props) Properties.storeToXML behaviour has changed from JDK 6 to 7

[Daniel Fuchs]

On 10/17/13 8:12 PM, Mandy Chung wrote:
> Daniel,
>
> Have you considered to revert to the previous version and simply iterate
> props.keySet rather than adding a new entry point just for
> jdk.internal.util.xml.PropertiesDefaultHandler to use?

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

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

-- daniel

>
> Mandy
>
> On 10/17/13 9:18 AM, Daniel Fuchs wrote:
>> Hi,
>>
>> Please find below a fix for:
>>
>> 8016344: (props) Properties.storeToXML behaviour has changed from
>>          JDK 6 to 7
>>
>> The code for storing properties to XML has changed significantly
>> between 7 & 8 - but the bug is still present in 8...
>>
>> This is the fix for JDK 8.
>>
>> By default Properties.storeToXML uses
>> sun.util.xml.PlatformXmlPropertiesProvider, and so the test
>> contains an explicit test for the alternate provider
>> (jdk.internal.util.xml.PropertiesDefaultHandler) which is only
>> used on profiles where the default provider is not available.
>>
>> AFAICT the test should run on all 3 profiles.
>>
>> <http://cr.openjdk.java.net/~dfuchs/webrev_8016344/webrev.00/>
>>
>> best regards,
>>
>> -- daniel
>