On 12/06/2012 09:38 AM, Doug Lea wrote:
Much better; thanks!
As a minor matter, it probably makes sense to give that class a
name. Even though it is a non-public static class, the name
will be seen here and there by debuggers, stacktraces, etc,
and the default "$" name will be confusing. As in:


static final class SuppliedThreadLocal<T> extends ThreadLocal<T> ..

public static <T> ThreadLocal<T> withInitial(Supplier<? extends T>
supplier) {
    return new SuppliedThreadLocal<T>(supplier);
}

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

Thanks

Reply via email to