<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 15:37:53 UTC 2016
On 17.10.16 17:39, Semyon Sadetsky wrote:
>
>
> On 17.10.2016 17:19, Sergey Bylokhov wrote:
>> On 17.10.16 15:16, Semyon Sadetsky wrote:
>>>> >>> 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.
>>> But I didn't get why it is risky? Usually find&replace works reliably.
>>
>> Because they works differently(this is why the second getModifiersEx
>> was added) and some code can rely on the behavior of getModifiers().
> At least usages of Event.*_MASK could be safely replaced with
> InputEvent.*_DOWN_MASK.
How it could be safe? both are a different constants which should be
used in pair with different methods?
> 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