<AWT Dev> [9] Review Request: 8038765 [macosx] Toolkit.sync should be implemented

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Tue Apr 8 09:53:16 UTC 2014


On 4/8/14 12:49 PM, Petr Pchelko wrote:
> Hello, Sergey.
>
> Why don't you add a boolean parameter to syncNativeQueue to avoid adding a new native method?
Two reasons:
  - syncNativeQueue will be reworked in the future, since it does not 
work correctly in all cases now.
  - If syncNativeQueue() will be reused, when I'll have to ignore its 
timeout parameter, this can be misleading
>
> With best regards. Petr.
>
> On 07.04.2014, at 23:23, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
>
>> Hello.
>> Please review the fix for jdk 9.
>> In the fix Toolkit.sync() is implemented in the same way as on other platforms. Note that on OSX we need additional step to flush the native selectors queue, because all our rendering was done to offscreen texture.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8038765
>> Webrev can be found at: http://cr.openjdk.java.net/~serb/8038765/webrev.00
>>
>> -- 
>> Best regards, Sergey.
>>


-- 
Best regards, Sergey.



More information about the awt-dev mailing list