Re: AWT Dev AWT [9] Review Request: JDK-8076151 Fix for Test java/awt/FontClass/CreateFont/fileaccess/FontFile.java fails

2015-04-08 Thread pooja chopra
Hi , Gentle Reminder. Please review below fix : 8076151 [TEST_BUG] Test java/awt/FontClass/CreateFont/fileaccess/FontFile.java fails Test bug fix. https://bugs.openjdk.java.net/browse/JDK-8076151 The webrev is: http://cr.openjdk.java.net/~kshefov/8076151/webrev.01/ Regards, Pooja On

Re: AWT Dev AWT [9] Review Request: JDK-8076151 Fix for Test java/awt/FontClass/CreateFont/fileaccess/FontFile.java fails

2015-04-01 Thread Alexander Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 3/31/2015 8:23 PM, pooja chopra wrote: Hi Alexander, I have updated the copyright year in TestFontFile.sh . Please find the updated webrev link :- the webrev is: http://cr.openjdk.java.net/~kshefov/8076151/webrev.01 Regards, Pooja On

Re: AWT Dev AWT [9] Review Request: JDK-8076151 Fix for Test java/awt/FontClass/CreateFont/fileaccess/FontFile.java fails

2015-03-31 Thread pooja chopra
Hi Alexander, I have updated the copyright year in TestFontFile.sh . Please find the updated webrev link :- the webrev is: http://cr.openjdk.java.net/~kshefov/8076151/webrev.01 Regards, Pooja On 3/30/2015 9:01 PM, Alexander Scherbatiy wrote: The fix looks good to me. Just update the

Re: AWT Dev AWT [9] Review Request: JDK-8076151 Fix for Test java/awt/FontClass/CreateFont/fileaccess/FontFile.java fails

2015-03-30 Thread Alexander Scherbatiy
The fix looks good to me. Just update the copyright year in the TestFontFile.sh. Thanks, Alexandr. On 3/27/2015 8:33 PM, pooja chopra wrote: Hello, Please review a fix for issue : 8076151 [TEST_BUG] Test java/awt/FontClass/CreateFont/fileaccess/FontFile.java fails Test bug fix.

AWT Dev AWT [9] Review Request: JDK-8076151 Fix for Test java/awt/FontClass/CreateFont/fileaccess/FontFile.java fails

2015-03-27 Thread pooja chopra
Hello, Please review a fix for issue : 8076151 [TEST_BUG] Test java/awt/FontClass/CreateFont/fileaccess/FontFile.java fails Test bug fix. https://bugs.openjdk.java.net/browse/JDK-8076151 The webrev is: http://cr.openjdk.java.net/~kshefov/8076151/webrev.00/ Thanks, Pooja