<Swing Dev> <AWT Dev> [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Mon Oct 17 18:53:01 UTC 2016


On 17.10.16 21:47, Semyon Sadetsky wrote:
>> It can be replaced if it will be used in pair with getModifiersEx().
>> The old getModifiers() is also deprecated. And javadoc for
>> getModifiersEx() describes what and how constants should be used.
> Can you add link to getModifiersEx() to all those constants' specs?

I assume you mean the new _DOWN constants not the old?

>>
>>>>
>>>>>>> And, please, limit the length of the changed line in VMPanel.java
>>>>>>> to 80
>>>>>>> chars.
>>>>>>>>
>>>>>>>>>>> On 10/2/2016 4:53 PM, Sergey Bylokhov wrote:
>>>>>>>>>>>> Thanks for the comments.
>>>>>>>>>>>> The new version:
>>>>>>>>>>>> http://cr.openjdk.java.net/~serb/8143077/webrev.01
>>>>>>>>>>>> The specification of Event class and InputEvent.getModifiers()
>>>>>>>>>>>> are
>>>>>>>>>>>> updated.
>>>>>>>>>>>>
>>>>>>>>>>>> On 30.09.16 19:08, Jonathan Bluett-Duncan wrote:
>>>>>>>>>>>>> Hi Sergey,
>>>>>>>>>>>>>
>>>>>>>>>>>>> I'm not a reviewer, but after reading the deprecation
>>>>>>>>>>>>> messages in
>>>>>>>>>>>>> Event.java
>>>>>>>>>>>>>
>>>>>>>>>>>>>     * @deprecated It is recommended that {@code AWTEvent}
>>>>>>>>>>>>> class
>>>>>>>>>>>>> and
>>>>>>>>>>>>> its
>>>>>>>>>>>>>     subclasses
>>>>>>>>>>>>>     *             be used instead.
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> I get the impression they would read better without the
>>>>>>>>>>>>> redundant
>>>>>>>>>>>>> "class" in the middle, like so.
>>>>>>>>>>>>>
>>>>>>>>>>>>>     * @deprecated It is recommended that {@code AWTEvent} and
>>>>>>>>>>>>> its
>>>>>>>>>>>>> subclasses
>>>>>>>>>>>>>     *             be used instead.
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> Kind regards,
>>>>>>>>>>>>> Jonathan
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> On 30 September 2016 at 16:45, Sergey Bylokhov
>>>>>>>>>>>>> <Sergey.Bylokhov at oracle.com
>>>>>>>>>>>>> <mailto:Sergey.Bylokhov at oracle.com>>
>>>>>>>>>>>>> wrote:
>>>>>>>>>>>>>
>>>>>>>>>>>>>     Hello.
>>>>>>>>>>>>>
>>>>>>>>>>>>>     Please review the fix for jdk9.
>>>>>>>>>>>>>
>>>>>>>>>>>>>     This is request to deprecate the obsolete flags inside
>>>>>>>>>>>>> InputEvent.
>>>>>>>>>>>>>     This constants were marked as obsolete in jdk1.4 and were
>>>>>>>>>>>>> replaced
>>>>>>>>>>>>>     by the new one. In jdk1.4 the documentation were update
>>>>>>>>>>>>> with
>>>>>>>>>>>>> notion
>>>>>>>>>>>>>     that the new constants should be used. And this bug is
>>>>>>>>>>>>> about
>>>>>>>>>>>>>     official deprecation of them.
>>>>>>>>>>>>>
>>>>>>>>>>>>>     We can replace old constants by the new one in the whole
>>>>>>>>>>>>> jdk,
>>>>>>>>>>>>> but I
>>>>>>>>>>>>>     updated only the code in InputEvent to make change
>>>>>>>>>>>>> smaller and
>>>>>>>>>>>>>     safer. So at least the new code in jdk and the code in
>>>>>>>>>>>>> applications
>>>>>>>>>>>>>     will start to use the new constants.
>>>>>>>>>>>>>
>>>>>>>>>>>>>     The changes in jconsole are necessary to fix deprecation
>>>>>>>>>>>>> warning.
>>>>>>>>>>>>>
>>>>>>>>>>>>>     jprt build passed, no new issues were found by jck/jtreg
>>>>>>>>>>>>> tests.
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>     Bug: https://bugs.openjdk.java.net/browse/JDK-8143077
>>>>>>>>>>>>> <https://bugs.openjdk.java.net/browse/JDK-8143077>
>>>>>>>>>>>>>     Webrev can be found at:
>>>>>>>>>>>>> http://cr.openjdk.java.net/~serb/8143077/webrev.00
>>>>>>>>>>>>> <http://cr.openjdk.java.net/~serb/8143077/webrev.00>
>>>>>>>>>>>>>
>>>>>>>>>>>>>     --
>>>>>>>>>>>>>     Best regards, Sergey.
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>
>>>>
>>>
>>
>>
>


-- 
Best regards, Sergey.



More information about the swing-dev mailing list