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

Daniel Fuchs daniel.fuchs at oracle.com
Wed May 6 12:44:47 UTC 2015


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



More information about the core-libs-dev mailing list