<AWT Dev> [9] Review Request: 8143054 [macosx] KeyEvent modifiers do not contain information about mouse buttons
Semyon Sadetsky
semyon.sadetsky at oracle.com
Thu Dec 24 17:21:40 UTC 2015
On 12/24/2015 7:37 PM, Phil Race wrote:
> On 12/24/2015 07:52 AM, Sergey Bylokhov wrote:
>>
>>> We use to add those fields usually but I don't see any obstacles to
>>> make
>>> the test anonymous. :)
>>
>> The author of the test(and of any other files) can be tracked by the
>> hg history.
>
> As someone who has maintained a lot of JDK code over the years I have
> seen
> files completely transformed such that the "author" had no hand in
> what is left
> except for the name of the file/class.
>
> So I think tracking via 'hg' is much more honest about who did what than
> asserting something about an entire file for all posterity and I would
> not
> add @author .. and don't think I ever have.
>
> -phil.
I did know about this new convention. I just wondered because the most
tests I saw had such fields... Still no objections.
The updated fix looks good to me.
--Semyon
More information about the awt-dev
mailing list