On 12/06/2012 07:29 PM, Doug Lea wrote:
On 12/06/12 13:00, Akhil Arora wrote:

http://cr.openjdk.java.net/~akhil/8003246.2/webrev/

- added SuppliedThreadLocal inner class
- reverted javadoc changes to no-args constructor
- added null check for Supplier


Great! Thanks for the quick response.

-Doug



Yes, great !

just nitpicking,
   return new SuppliedThreadLocal<T>(supplier);
should be
   return new SuppliedThreadLocal<>(supplier);

Also, can you add a @see in the constructor of ThreadLocal that references ThreadLocal.withInitialValue(Supplier).

cheers,
Rémi

Reply via email to