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]>; [email protected] Cc: Langer, Christoph <[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
