Re: RFR : 8211317: avoid memory leak in Java_sun_awt_UNIXToolkit_load_1stock_1icon

Lindenmaier, Goetz Tue, 09 Oct 2018 00:25:08 -0700

Hi Matthias,

You can push this now.  There are two Reviewers,
the change sat there more than 24h, and it’s a simple fix.

Thanks, Goetz

From: Langer, Christoph
Sent: Monday, October 8, 2018 10:01 AM
To: Baesken, Matthias <[email protected]>; Lindenmaier, Goetz 
<[email protected]>; [email protected]
Subject: RE: RFR : 8211317: avoid memory leak in 
Java_sun_awt_UNIXToolkit_load_1stock_1icon

Hi Matthias,

+1

Best regards
Christoph

From: Baesken, Matthias
Sent: Donnerstag, 4. Oktober 2018 09:26
To: Lindenmaier, Goetz 
<[email protected]<mailto:[email protected]>>; 
[email protected]<mailto:[email protected]>
Cc: Langer, Christoph 
<[email protected]<mailto:[email protected]>>
Subject: RE: RFR : 8211317: avoid memory leak in 
Java_sun_awt_UNIXToolkit_load_1stock_1icon

Thanks Goetz !

As far as I know a second review is needed  before pushing –  waiting for the 
second review 😉 …

From: Lindenmaier, Goetz
Sent: Donnerstag, 4. Oktober 2018 09:04
To: Baesken, Matthias 
<[email protected]<mailto:[email protected]>>; 
[email protected]<mailto:[email protected]>
Cc: Langer, Christoph 
<[email protected]<mailto:[email protected]>>
Subject: RE: RFR : 8211317: avoid memory leak in 
Java_sun_awt_UNIXToolkit_load_1stock_1icon

Hi Matthias,

good catch, thanks for fixing this.
Looks good.

Best regards,
  Goetz.

From: Baesken, Matthias
Sent: Donnerstag, 4. Oktober 2018 08:45
To: [email protected]<mailto:[email protected]>
Cc: Lindenmaier, Goetz 
<[email protected]<mailto:[email protected]>>; Langer, 
Christoph <[email protected]<mailto:[email protected]>>
Subject: RE: RFR : 8211317: avoid memory leak in 
Java_sun_awt_UNIXToolkit_load_1stock_1icon

Hello could I have a review for this small change ?

Thanks, Matthias

From: Baesken, Matthias
Sent: Montag, 1. Oktober 2018 12:19
To: [email protected]<mailto:[email protected]>
Subject: RFR : 8211317: avoid memory leak in 
Java_sun_awt_UNIXToolkit_load_1stock_1icon

Hello ,  please review this small change .

It adds missing resource – freeing to        
Java_sun_awt_UNIXToolkit_load_1stock_1icon  .

Bug/webrev :

https://bugs.openjdk.java.net/browse/JDK-8211317

http://cr.openjdk.java.net/~mbaesken/webrevs/8211317.0/

Best regards, Matthias

Re: RFR : 8211317: avoid memory leak in Java_sun_awt_UNIXToolkit_load_1stock_1icon

Reply via email to