<AWT Dev> RFR : 8211317: avoid memory leak in Java_sun_awt_UNIXToolkit_load_1stock_1icon

[Langer, Christoph]

Hi Matthias,

+1

Best regards
Christoph

From: Baesken, Matthias
Sent: Donnerstag, 4. Oktober 2018 09:26
To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; awt-dev at openjdk.java.net
Cc: Langer, Christoph <christoph.langer at sap.com>
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 <matthias.baesken at sap.com<mailto:matthias.baesken at sap.com>>; awt-dev at openjdk.java.net<mailto:awt-dev at openjdk.java.net>
Cc: Langer, Christoph <christoph.langer at sap.com<mailto:christoph.langer at sap.com>>
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: awt-dev at openjdk.java.net<mailto:awt-dev at openjdk.java.net>
Cc: Lindenmaier, Goetz <goetz.lindenmaier at sap.com<mailto:goetz.lindenmaier at sap.com>>; Langer, Christoph <christoph.langer at sap.com<mailto:christoph.langer at sap.com>>
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: awt-dev at openjdk.java.net<mailto:awt-dev at openjdk.java.net>
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20181008/93eeb5b9/attachment.html>