Review request: JDK8 b91 source with GPL header errors

[David Katleman]

On 5/28/2013 8:50 PM, David Holmes wrote:
> On 29/05/2013 1:33 PM, Mike Duigou wrote:
>>
>> On May 28 2013, at 18:45 , David Holmes wrote:
>>
>>> On 29/05/2013 11:35 AM, David Katleman wrote:
>>>> Below are corrections to GPL headers found in JDK8 b91
>>>>
>>>>      http://cr.openjdk.java.net/~katleman/8015525/webrev
>>>>
>>>> All are punctuation issues, no code is involved, anyone can review.
>>>>
>>>> As a veteran of many such header reviews, reviewing the patch link at
>>>> the top of the webrev is often easier than each individual file.
>>>
>>> All good.
>>>
>>> Will we never get to the end of this saga :(
>>
>> Davids,
>>
>> I have a partial solution and also the explanation for the missing 
>> commas in the lambda test files:
>>
>> The templates in make/templates don't include a comma after the 
>> %YEARS% variable. Can we add one to the templates? These templates 
>> are what I have been using and telling others to use. Since I am 
>> guilty of pushing some of these without the comma I know, at least, 
>> those would have been prevented.
>
> There is no specification for the format of the %YEARS% variable, so 
> rather than showing a comma after %YEARS% shouldn't something specify 
> that %YEARS% is either:
>
> MMMM,
>
> or
>
> MMMM, NNNN,
>
> else we risk users of the template using eg 2006-2013

Good points.

%YEARS% was added to the template back in Sun days, where the two valid 
formats were MMMM or NNNN-MMMM with no commas in sight.

I've opened a CR to track such an update, once we come up with one.

     https://jbs.oracle.com/bugs/browse/JDK-8015626

How about

         YEAR, [END_YEAR,]

                     Dave