RFR: JDK-8074002 java.time.ZoneId.systemDefault() should be faster

[Daniel Fuchs]

Hi Peter,

The logic looks good to me.
But I'm not an expert of the field, despite my small incursions :-)

I wish we didn't have to do defensive cloning - but I don't see
any way around.


best regards,

-- daniel

On 27/04/15 17:26, Peter Levart wrote:
> Hi,
>
> Please review the following improvement that caches default ZoneId
> object and makes the frequently executed ZoneId.systemDefault() method
> faster:
>
> http://cr.openjdk.java.net/~plevart/jdk9-dev/ZoneId.systemDefault/webrev.04/
>
>
> The patch is just a rebased version of webrev.03 + some comments added
> about the importance of defensive cloning in TimeZone.setDefault().
> There was already a discussion about this patch a while ago and Stephen
> basically approved it in this form:
>
> http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031714.html
>
>
>
> Thanks,
>
> Peter Levart