RFR: 8214552: Resolve clash between 4947890 and 8130266

[Phil Race]

Sorry, but its actually  8130264 that I meant

https://bugs.openjdk.java.net/browse/JDK-8130264

-phil.

On 11/30/18 11:54 AM, Mandy Chung wrote:
>
>
> On 11/30/18 11:43 AM, Phil Race wrote:
>> Roger,
>>
>> I just pushed 8130266 which had the CSR approved a couple of days ago 
>> only
>> to find that it clashed badly with the change you made in 4947890 to 
>> reduce
>> the cost of system property initialisation.
>> It appears that the new code there is not happy if there is no value 
>> for the
>> system property.
>>
>
> Is JDK-8214322 the JBS issue but not JDK-8130266?
>
>> I think java.awt.printerjob just needs to be removed from the new code :
>>
>> If this looks right to you I need to push ASAP :
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8214552
>> Webrev: http://cr.openjdk.java.net/~prr/8214552/
>>
>
> src/java.base/share/native/libjava/System.c also needs to be updated.
>
> Mandy
>
>> Separately I think the SystemProps code ought to consider whether
>> it ought to error out the launcher if a property is not defined ..
>>
>> -phil.
>>
>>
>