<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 14:19:19 UTC 2016
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().
>>> 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