<Swing Dev> <AWT Dev> [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK
Semyon Sadetsky
semyon.sadetsky at oracle.com
Mon Oct 17 18:47:35 UTC 2016
On 10/17/2016 9:23 PM, Sergey Bylokhov wrote:
> On 17.10.16 21:14, Semyon Sadetsky wrote:
>> Then this explanation should be added to the javadoc deprecation note
>> because currently it states that those constants can be replaced with
>> the new ones. But actually they are related to different APIs and cannot
>> simply substitute each other.
>
> 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?
>
>>>
>>>>>> 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.
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>
>>>
>>>
>>
>
>
More information about the swing-dev
mailing list