<AWT Dev>  Review request: 8041987 [macosx] setDisplayMode crashes
anthony.petrov at oracle.com
Mon Apr 28 12:19:31 UTC 2014
The fix looks fine.
On 4/28/2014 1:50 PM, Petr Pchelko wrote:
> Hello, AWT Team.
> Please review the fix for the issue:
> The fix is available at:
> My recent fix with memory leak uncovered one more issue: in nativeSetDisplayMode the reference obtained outside of the block we use in the block.
> That's fine for Objective-C pointers, because they are auto-retained and autoreleased in this case. But for core foundation types we need to retain/release
> them manually. I've run all GraphicsDevice/GraphicsConfiguration/Fullscreen tests. I've also briefly skimmed the codebase for similar issues, but didn't find
> any obvious problems of such kind.
> Also I'm open-sourcing the tests which found the problem.
> With best regards. Petr.
More information about the awt-dev