RFR: 8214552: Resolve clash between 4947890 and 8130266

[Roger Riggs]

Hi Phil,

The patch looks fine.

Where does it die?  SystemProps should skip null values in putIfAbsent().

Sorry for the turbulence.

Roger



On 11/30/2018 02:43 PM, 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.
>
> 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/
>
> Separately I think the SystemProps code ought to consider whether
> it ought to error out the launcher if a property is not defined ..
>
> -phil.
>
>