Code Review for WeakReference leak in the Logging API (6942989)

Daniel D. Daugherty daniel.daugherty at oracle.com
Mon Jun 14 09:46:20 PDT 2010


On 6/11/2010 4:41 PM, Martin Buchholz wrote:
> On Fri, Jun 11, 2010 at 14:46, Daniel D. Daugherty
> <daniel.daugherty at oracle.com> wrote:
>   
>> Jeremy,
>>
>> I'm definitely interested in learning about your approach to this issue.
>>     
>
> Here's the patch against openjdk6 by Jeremy.
> http://cr.openjdk.java.net/~martin/WeakLogger-jeremymanson.patch
> (It would take a bit of merging to port to openjdk7)
>
> Feel free to take anything from our change.
> Apologies for the perforce-isms.
>
> Martin
>   

Jeremy and Martin,

Thanks for the proposed fix. A couple of questions:

- This changeset is private to Google right now, correct? As in
  it hasn't made it into OpenJDK6 yet.
- Do you plan on pushing this changeset to OpenJDK6?
- What kind of testing has been done on it?

Thanks for the offer for the code. I'll start wading through the
diffs today. Because this is an escalated issue, I will likely
be taking just the code and comments directly related to the
problem at hand. The JavaDoc fixes, even though they are useful,
will have to wait for a different changeset.

Thanks for jumping in on this thread.

Dan



More information about the serviceability-dev mailing list