[9] Review request for 8175025: The copyright section in the test/java/awt/font/TextLayout/DiacriticsDrawingTest.java should be updated

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[9] Review request for 8175025: The copyright section in the test/java/awt/font/TextLayout/DiacriticsDrawingTest.java should be updated

Dmitry Markov
Hello,

Could you review a simple fix for jdk9, please?

        bug: https://bugs.openjdk.java.net/browse/JDK-8175025
        webrev: http://cr.openjdk.java.net/~dmarkov/8175025/webrev.00/

Changed the license in the test to GPL v2.

Thanks,
Dmitry
Reply | Threaded
Open this post in threaded view
|

Re: [9] Review request for 8175025: The copyright section in the test/java/awt/font/TextLayout/DiacriticsDrawingTest.java should be updated

Sergey Bylokhov
Looks fine.

>
> Hello,
>
> Could you review a simple fix for jdk9, please?
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8175025
> webrev: http://cr.openjdk.java.net/~dmarkov/8175025/webrev.00/
>
> Changed the license in the test to GPL v2.
>
> Thanks,
> Dmitry

Reply | Threaded
Open this post in threaded view
|

Re: [9] Review request for 8175025: The copyright section in the test/java/awt/font/TextLayout/DiacriticsDrawingTest.java should be updated

Vadim Pakhnushev
+1

Vadim

On 16.02.2017 15:13, Sergey Bylokhov wrote:

> Looks fine.
>
>> Hello,
>>
>> Could you review a simple fix for jdk9, please?
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8175025
>> webrev: http://cr.openjdk.java.net/~dmarkov/8175025/webrev.00/
>>
>> Changed the license in the test to GPL v2.
>>
>> Thanks,
>> Dmitry