<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