<AWT Dev>  Review Request for 8009911 : [macosx] SWT app freeze when going full screen using Java 7 on Mac
joe.darcy at oracle.com
Wed May 22 05:33:52 UTC 2013
Yes; Kumar should be one of the reviewers here.
On 05/21/2013 04:18 AM, Anthony Petrov wrote:
> Hi Petr,
> The fix looks good to me. We still need a review from someone who
> maintains the Launcher code. Kumar?
> Also, you could use the @autoreleasepool directive to simplify it a
> bit, although I'm fine with the explicit syntax, too.
> One comment regarding your remark:
>> c. We would have lost all the hg history for this file.
> We wouldn't. Mercurial handles renaming of files very well. See `hg
> help rename` for more details. (I'm not suggesting to rename this file
> best regards,
> On 05/21/2013 10:57 AM, David Holmes wrote:
>> Adding core-libs as this is actually a launcher change.
>> On 21/05/2013 4:36 PM, Petr Pchelko wrote:
>>> Hello, AWT Team.
>>> Please review the fix for the issue:
>>> The webrev is available at:
>>> The problem:
>>> When launching java with -XstatrOnFirstThread we were using
>>> dispatch_sync to start a Java main on the main thread. This blocked
>>> the main dispatch queue, so all the subsequent calls to GCD were never
>>> invoked. GCD is used a bit in our code, but it is used inside Cocoa,
>>> for example for fullscreen.
>>> The solution:
>>> We now launch Java main using a performSelectorOnMaintThread, so we do
>>> not block GCD.
>>> 1. This also fixes the following SWT bug:
>>> 2. Although I have added OBJC syntax into java_md_macosx.c file, I did
>>> not rename it to .m, because:
>>> a. It is hard to update an old build system to compile with the
>>> new file name.
>>> b. It is already compiled with -x objective-c, so I will compile
>>> c. We would have lost all the hg history for this file.
>>> With best regards. Petr.
More information about the core-libs-dev