Daniel D. Daugherty wrote:
:
Here is the URL for the webrev:
http://cr.openjdk.java.net/~dcubed/6942989-webrev/1/
Thanks, in advance, for any reviews.
Sorry for the late reply. Using the reference queue is much better. I've
looked through the new webrev and the approach seems reasonable.
One nit is that the style is perhaps a bit different to the existing
code. For example, the alignment of the declarations in LoggerWeakRef is
different to the enclosing class and the other classes in this package,
the new package-private methods are final whereas the existing
package-private methods aren't. Nothing wrong, just looks a bit different.
One question, in drainLoggerRefQueueBounded I'm curious about the check
for loggerRefQueue being null. Is that needed?
-Alan.
