On 11/5/2014 8:15 AM, Daniel Fuchs wrote:
Here is the new webrev:http://cr.openjdk.java.net/~dfuchs/webrev_8060132/webrev.03/index.html
Looks good. Minor nits and no need to generate new webrev.
line 108: {@linkplain #reset} -- # is missing.
line 918-921: nit: formatting looks a little inconsistent with the code
around it.
Perhaps better to wrap them in 2 lines:
boolean ensureCloseOnReset = names.length > 0 &&
getBooleanProperty(handlersPropertyName + ".ensureCloseOnReset", true);
Mandy
