Mikhail: I've just pushed your fix to the repository:
http://hg.openjdk.java.net/jdk8/awt/jdk/rev/f42d178f0452
--
best regards,
Anthony
On 10/16/2012 3:51 PM, Sergey Bylokhov wrote:
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.
