Proposed revised format for JDK commit messages

[Magnus Ihse Bursie]

/Magnus

> 19 sep. 2018 kl. 09:20 skrev Per Liden <per.liden at oracle.com>:
> 
> Hi,
> 
>> On 09/17/2018 06:27 AM, joe darcy wrote:
>> [...]
>>   ReviewersLine : "Reviewed-by: " Username (", " Username)* "\n"
>>   ContributedByLine : "Contributed-by: " Text
> [...]
>> Proposed:
>>     8167108: inconsistent handling of SR_lock can lead to crashes
>>     Add Thread Safe Memory Reclamation (Thread-SMR) mechanism.
>>     Reviewed-by: coleenp, dcubed, dholmes, eosterlund, gthornbr, kbarrett, rehn, sspitsyn, stefank
>>     Co-authored-by: Erik Osterlund <erik.osterlund at oracle.com>
>>     Co-authored-by: Robbin Ehn <robbin.ehn at oracle.com>
>>     Co-authored-by: Igor Ignatyev <igor.ignatyev at oracle.com>
> 
> It would feel natural to have all *-by lines follow the same syntax. The Reviewed-by line sticks out in this regard. Is there a reason why we don't just do:
> 
> Reviewed-by: Some Name <some.name at oracle.com>
> Reviewed-by: Some Other Name <some.other.name at oracle.com>
> 
> ... and so on?

Per,

This is just as the case is today. The Contributed-by follows different rules than Reviewed-by, since anyone can contribute a patch, but only OpenJDK Reviewers can review a patch. 

/Magnus 

> 
> cheers,
> Per