<AWT Dev> [8] [PATCH] 6818083: When DISPLAY is bad, InternalError thrown, not AWTError

[Sergey Bylokhov]

Hi, Mikhail.
Fix looks good.

16.10.2012 15:48, Anthony Petrov wrote:
> Thanks Mikhail. I've published the updated patch as a webrev at:
>
> http://cr.openjdk.java.net/~anthony/8-48-badDisplay-6818083.0/
>
> All: could anyone else please review it as well?
>
> -- 
> best regards,
> Anthony
>
> On 10/15/2012 8:07 PM, Mikhail Cherkasov wrote:
>> Test was renamed to BadDisplayTest.
>> New patch is attached.
>>
>> On 10/15/2012 6:42 PM, Anthony Petrov wrote:
>>> Hi Mikhail,
>>>
>>> The fix looks good to me. However, please consider naming the test 
>>> files with human-readable names. E.g. 
>>> test/java/awt/Toolkit/BadDisplayTest/BadDisplayTest.java|.sh.
>>>
>>> -- 
>>> best regards,
>>> Anthony
>>>
>>> On 10/12/12 19:43, Mikhail Cherkasov wrote:
>>>> Hi all,
>>>>
>>>> Could you please review a fix for 6818083: When DISPLAY is bad,
>>>> InternalError thrown, not AWTError
>>>> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6818083
>>>>
>>>> InternalError was replaced with AWTError. Patch is attached.
>>>>
>>>> Thanks,
>>>> Mikhail.
>>


-- 
Best regards, Sergey.