<AWT Dev> [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms

[Shashidhara Veerabhadraiah]

Hi Sergey, Then the fix looks good to me.

Thanks and regards,
Shashi

-----Original Message-----
From: Sergey Bylokhov 
Sent: Friday, March 8, 2019 3:20 AM
To: Shashidhara Veerabhadraiah <shashidhara.veerabhadraiah at oracle.com>; Dmitry Markov <dmitry.markov at oracle.com>
Cc: awt-dev at openjdk.java.net
Subject: Re: <AWT Dev> [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms

Hi, Shashi.
On 07/03/2019 07:45, Shashidhara Veerabhadraiah wrote:
> Hi Sergey, I think you need to add "@run main" command and this bug id to the "@bug" list of bugs. Also you can update the copyright year to this year as well.

  - The "@run main" tag is optional, this is default action for the test.
  - The @bug tag contains a bugid for which the test was written or the bugid which was found by this test later. It can also contain the bugid of bugfix in the test if the fix rewrote the test.
  - I'll update the year before the push.

> 
> Thanks and regards,
> Shashi
> 
> -----Original Message-----
> From: Dmitry Markov
> Sent: Thursday, March 7, 2019 2:59 PM
> To: Sergey Bylokhov <sergey.bylokhov at oracle.com>
> Cc: awt-dev at openjdk.java.net
> Subject: Re: <AWT Dev> [13] Review Request: 8219504 Test for 
> JDK-8211435 can be run on all platforms
> 
> Hi Sergey,
> 
> The fix looks good to me.
> 
> Thanks,
> Dmitry
> 
>> On 7 Mar 2019, at 06:59, Sergey Bylokhov <sergey.bylokhov at oracle.com> wrote:
>>
>> Thank you for review!
>> Looking for the second +1 from someone else.
>>
>> On 20/02/2019 22:20, Krishna Addepalli wrote:
>>> +1
>>> Krishna
>>> -----Original Message-----
>>> From: Sergey Bylokhov
>>> Sent: Thursday, February 21, 2019 8:01 AM
>>> To: awt-dev at openjdk.java.net
>>> Subject: <AWT Dev> [13] Review Request: 8219504 Test for JDK-8211435 
>>> can be run on all platforms Hello.
>>> Please review the fix for jdk 13.
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8219504
>>> Fix: http://cr.openjdk.java.net/~serb/8219504/webrev.00
>>> Currently the test for JDK-8211435 is executed on macOS only, because of JDK-8204142[1][2]:
>>> But since JDK-8204142 was fixed we can run it on all platforms.
>>> [1] https://bugs.openjdk.java.net/browse/JDK-8204142
>>> [2]
>>> http://mail.openjdk.java.net/pipermail/awt-dev/2018-October/014492.h
>>> t
>>> ml
>>
>>
>> --
>> Best regards, Sergey.
> 


--
Best regards, Sergey.