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
- Previous message: RFR: JDK-8074002 java.time.ZoneId.systemDefault() should be faster
- Next message: RFR: JDK-8074002 java.time.ZoneId.systemDefault() should be faster
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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
- Previous message: RFR: JDK-8074002 java.time.ZoneId.systemDefault() should be faster
- Next message: RFR: JDK-8074002 java.time.ZoneId.systemDefault() should be faster
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]