<Swing Dev> <AWT> [9] Review Request: JDK-8076151 Fix for Test java/awt/FontClass/CreateFont/fileaccess/FontFile.java fails
Alexander Zvegintsev
alexander.zvegintsev at oracle.com
Wed Apr 8 10:27:57 UTC 2015
Hi Pooja,
Please fix the copyright header in TestFontFile.sh before the push, it
is too short(only 2 lines).
Otherwise the fix looks good to me.
Thanks,
Alexander.
On 04/08/2015 10:29 AM, pooja chopra wrote:
> 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 4/1/2015 2:28 PM, Alexander Scherbatiy wrote:
>>
>> 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 3/30/2015 9:01 PM, Alexander Scherbatiy wrote:
>>>>
>>>> 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.
>>>>> https://bugs.openjdk.java.net/browse/JDK-8076151
>>>>> The webrev is: http://cr.openjdk.java.net/~kshefov/8076151/webrev.01/
>>>>>
>>>>> Thanks,
>>>>> Pooja
>>>>
>>>
>>
>
More information about the swing-dev
mailing list