RFR: 8164052 Fix legal notices in JDK tests
Alexandre (Shura) Iline
alexandre.iline at oracle.com
Tue Aug 16 14:13:47 UTC 2016
> On Aug 16, 2016, at 7:11 AM, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
>
> On 16.08.16 17:00, Alexandre (Shura) Iline wrote:
>>
>>> On Aug 16, 2016, at 5:38 AM, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
>>>
>>> The are some issues when 2016 was replaced to "2016, 2016" for example:
>>> test/java/awt/Focus/Cause/FocusCauseTest.java
>>
>> In this file, originally incorrect “2016, 2016,” is replaced by correct “2016,” - this is a proper fix IMO. Are you suggesting that the same year could repeat twice?
>
> If it should be the one year then looks fine.
> This year was changed recently from 2016->2016,2016 in JDK-8159690, it seems it was a typo.
Just to close this, you said there are “some” issues. Are there others besides this file which do not look right?
Shura
>
>>>
>>> On 15.08.16 21:39, Alexandre (Shura) Iline wrote:
>>>> Hi,
>>>>
>>>> Please review this bulk update of JDK test sources which is fixing legal notices formatting and content.
>>>>
>>>> http://cr.openjdk.java.net/~shurailine/8164052/webrev_00/
>>>>
>>>> I am sending this to the jdk9-dev at openjdk.java.net hoping that would be enough. Otherwise I would send the request to multiple aliases which have tests in idk repository.
>>>>
>>>> Shura
>>>>
>>>
>>>
>>> --
>>> Best regards, Sergey.
>>
>
>
> --
> Best regards, Sergey.
More information about the jdk9-dev
mailing list