Request for review: 6799689

Joshua Bloch jjb at google.com
Thu Feb 26 01:50:47 UTC 2009


Mandy,
Hi! Looks good, but if you care about the performance of parseHexString, you
might want to use the "lazy initialization holder class idiom" (Effective
Java 2e, Page 283).  I can't promise that you'll be able to measure the
difference, or that it will be faster, but I suspect so.

    Josh

On Wed, Feb 25, 2009 at 5:27 PM, Mandy Chung <Mandy.Chung at sun.com> wrote:

> 6799689 Make sun.misc.FloatingDecimal.hexFloatPattern static field
> initialized lazily
>
> Webrev:
>   http://cr.openjdk.java.net/~mchung/6799689/webrev.00/
>
> The Pattern object is not always needed but is currently instantiated in
> the static initializer. Lazy initialization of the hexFloatFattern static
> field will save the regex classes not to be loaded until it's needed.
>
> Thanks
> Mandy
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/core-libs-dev/attachments/20090225/7456bc50/attachment.html>


More information about the core-libs-dev mailing list