On Dec 4, 2015, at 11:56 AM, Roger Riggs <roger.ri...@oracle.com> wrote: > > Hi Mandy, > > The webrev[1] and javadoc[2] are updated in to address your comments. > > Thanks, Roger > > [1] http://cr.openjdk.java.net/~rriggs/webrev-cleaner-8138696/ > [2] http://cr.openjdk.java.net/~rriggs/cleaner-doc/index.html >
One tiny nit: src/java.base/share/classes/jdk/internal/misc/CleanerImpl.java 136 // Cleanable the thread locals "Cleanable" seems like the wrong word here. All of my previous points have been addressed. Looks good to me.