On Wed, 21 Jul 2021 07:19:02 GMT, Sergey Bylokhov <s...@openjdk.org> wrote:

>> Phil Race has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   8268824: Remove unused jdk.accessibility APIs deprecated for removal in 
>> JDK 9
>
> src/jdk.accessibility/share/classes/com/sun/java/accessibility/util/AWTEventMonitor.java
>  line 53:
> 
>> 51:     public AWTEventMonitor() {}
>> 52: 
>> 53:     static private Component componentWithFocus_private = null;
> 
> Can we rename the "new" "xx_private" fields to the shorter version? The 
> javadoc for the protected fields can be applied to the "private version"(but 
> w/o @Deprecated tag).
> 
> So in fact we can change "protected" to the "private", drop the "@Deprecated" 
> tags, and delete the "xx_private" versions of the fields.

I don't think the javadoc is needed since these are now private fields.
I can however rename them to remove _private.

-------------

PR: https://git.openjdk.java.net/jdk/pull/4845

Reply via email to