RFR (XXS): 8129602: Incorrect GPL header causes RE script to create wrong output

[Coleen Phillimore]

Thomas,

This looks good (in case you haven't already pushed it).  I think you 
fixed your removal of the extra blank lines in the final 
http://cr.openjdk.java.net/~tschatzl/8129602/webrev.8u60/hotspot.patch 
patch.  I'm sorry I didn't address all of these issues when I did the 
one assigned to me.   David pointed the other bugs out to me after I'd 
started my JPRT job.

Now we know something will catch missing , at the end of copyright years.

Coleen

On 6/24/15 5:58 AM, David Holmes wrote:
> Just saw your follow up mail so all is good.
>
> Thanks,
> David
>
> On 24/06/2015 7:56 PM, David Holmes wrote:
>> Hi Thomas,
>>
>> On 24/06/2015 7:03 PM, Thomas Schatzl wrote:
>>> Hi all,
>>>
>>>    I need reviews for a small set of fixes that removes superfluous
>>> commas after the year in the GPL header so that some release 
>>> engineering
>>> script does not produce garbage.
>>
>> Actually add the missing commas where needed :)
>>
>>> This CR fixes these issues for several files across the hotspot
>>> repository, bundling multiple similar problems. That's why this issue
>>> has been sent for review to hotspot-dev.
>>>
>>> CR (confidential, sorry):
>>> https://bugs.openjdk.java.net/browse/JDK-8129602
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~tschatzl/8129602/webrev.hotspot/
>>
>> You need to look at the patch to see the whitespace fix :)
>>
>>> http://cr.openjdk.java.net/~tschatzl/8129602/webrev.hs-gc/ (for
>>> whitebox.cpp).
>>
>> Ditto.
>>
>>> Here is the corresponding 8u60 patch that differs in paths and adds a
>>> few more typos that have been fixed in 9.
>>>
>>> http://cr.openjdk.java.net/~tschatzl/8129602/webrev.8u60/
>>
>> This all looks fine too. So the commas are already fixed in 9?
>>
>> Thanks,
>> David
>>
>>> Thanks,
>>>    Thomas
>>>