<AWT Dev> Review request for CR 2227234 - [macosx] Exception at java.awt.datatransfer on headless mode (only in GUI session)

Anthony Petrov anthony.petrov at oracle.com
Fri Jul 19 06:08:41 PDT 2013


The fix looks good to me.

--
best regards,
Anthony

On 07/18/2013 03:20 PM, Petr Pchelko wrote:
> Hello, AWT team.
>
> It's a reminder about a very old review I have posted.
>
>> Could you please review a fix for the following issue:
>> http://bugs.sun.com/view_bug.do?bug_id=2227234
>> The fix is available at:
>> http://cr.openjdk.java.net/~pchelko/2227234/webrev.00/
>> 	
>> This is a partial forward port of the fix made by Alexander Zyev in jdk7.
>> The original bug in 7 is:
>> http://bugs.sun.com/view_bug.do?bug_id=7184951
>> The original fix could be found at:
>> http://cr.openjdk.java.net/~kizune/7184951/webrev.00/
>>
>> The changes in CToolkitThreadBlockedHandler are not needed after the fix for
>> JDK-8006634 Unify LWCToolkit.invokeAndWait() and CToolkitThreadBlockedHandler
>
> Artem has already approved the fix, could I get a second review please?
>
> With best regards. Petr.
>
> On May 16, 2013, at 1:11 PM, Artem Ananiev wrote:
>
>>
>> Looks fine.
>>
>> Thanks,
>>
>> Artem
>>
>> On 5/16/2013 12:50 PM, Petr Pchelko wrote:
>>> Hello, AWT Team.
>>>
>>> This is a reminder. Could you please review this fix.
>>>
>>> With best regards. Petr.
>>>
>>> On Mar 14, 2013, at 2:54 PM, Petr Pchelko wrote:
>>>
>>>> Hello, AWT Team.
>>>>
>>>> Could you please review a fix for the following issue:
>>>> http://bugs.sun.com/view_bug.do?bug_id=2227234
>>>> The fix is available at:
>>>> http://cr.openjdk.java.net/~pchelko/2227234/webrev.00/
>>>> 	
>>>> This is a partial forward port of the fix made by Alexander Zyev in jdk7.
>>>> The original bug in 7 is:
>>>> http://bugs.sun.com/view_bug.do?bug_id=7184951
>>>> The original fix could be found at:
>>>> http://cr.openjdk.java.net/~kizune/7184951/webrev.00/
>>>>
>>>> The changes in CToolkitThreadBlockedHandler are not needed after the fix for
>>>> JDK-8006634 Unify LWCToolkit.invokeAndWait() and CToolkitThreadBlockedHandler
>>>>
>>>> With best regards. Petr.
>>>>
>>>
>


More information about the awt-dev mailing list