Re: RFR 8233958: Memory retention due to HttpsURLConnection finalizer that serves no purpose

2019-11-12 Thread Roger Riggs
+1, Another finalizer gone! On 11/12/19 12:57 PM, Daniel Fuchs wrote: Looks good to me Michael. Good riddance! :-) best regards, -- daniel On 12/11/2019 16:45, Michael McMahon wrote: Hi, Could I get the following small change reviewed please? 8233958 complains about memory retention caus

Re: RFR 8233958: Memory retention due to HttpsURLConnection finalizer that serves no purpose

2019-11-12 Thread Daniel Fuchs
Looks good to me Michael. Good riddance! :-) best regards, -- daniel On 12/11/2019 16:45, Michael McMahon wrote: Hi, Could I get the following small change reviewed please? 8233958 complains about memory retention caused by a finalizer which does nothing, other than delay garbage collection

RFR 8233958: Memory retention due to HttpsURLConnection finalizer that serves no purpose

2019-11-12 Thread Michael McMahon
Hi, Could I get the following small change reviewed please? 8233958 complains about memory retention caused by a finalizer which does nothing, other than delay garbage collection of the associated objects. The proposed change is to remove the dead code. http://cr.openjdk.java.net/~michaelm/823