Re: [8] Review request for JI-9038899: [macosx]Memory leak in ScreenMenu

On 6/22/2016 8:59 PM, Alexandr Scherbatiy wrote: On 6/22/2016 6:14 PM, Robin Stevens wrote: Hello all, my OCA has just been accepted. Let me know if there are any other steps I need to take to get this review started. The issue should be fixed in JDK 9 first. Could you prepare a webrev

Re: [8] Review request for JI-9038899: [macosx]Memory leak in ScreenMenu

On 6/22/2016 6:14 PM, Robin Stevens wrote: Hello all, my OCA has just been accepted. Let me know if there are any other steps I need to take to get this review started. The issue should be fixed in JDK 9 first. Could you prepare a webrev [1] for the fix against the JDK 9 client

Re: [8] Review request for JI-9038899: [macosx]Memory leak in ScreenMenu

Hello all, my OCA has just been accepted. Let me know if there are any other steps I need to take to get this review started. Kind regards, Robin On Wed, Jun 8, 2016 at 9:17 AM, Robin Stevens wrote: > Hello all, > > are there any additional steps I need to take to get a

Re: [8] Review request for JI-9038899: [macosx]Memory leak in ScreenMenu

Hello all, are there any additional steps I need to take to get a review of this patch started ? Kind regards, Robin On Tue, May 31, 2016 at 5:01 PM, Robin Stevens wrote: > Hello, > > I created a patch for a bug I just logged through http://bugs.java.com/ > (still under

[8] Review request for JI-9038899: [macosx]Memory leak in ScreenMenu

Hello, I created a patch for a bug I just logged through http://bugs.java.com/ (still under review with identifier JI-9038899). The com.apple.laf.ScreenMenu class keeps hard references to JMenuItems which have been removed from the JMenu. The patch contains a fix for the memory leak and a test