RFR: 8303175: (fs) Deprecate com.sun.nio.file.SensitivityWatchEventModifier for removal [v4]

[Alan Bateman]

On Wed, 1 Mar 2023 18:23:43 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

>> Deprecate `SensitivityWatchEventModifier` for now instead of directly removing it as proposed in #12626.
>
> Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8303175: Change @deprecated verbiage in SensitivityWatchEventModifier

src/jdk.unsupported/share/classes/com/sun/nio/file/SensitivityWatchEventModifier.java line 41:

> 39:  * which used this {@code WatchEvent.Modifier}.  Therefore this
> 40:  * {@code WatchEvent.Modifier} class is vestigial and should be removed in a
> 41:  * future release.

"no longer recognizes" suggests that using them will lead to an error but it's actually that they will be just ignored. 

Also "should be removed in a future release" will duplicate text generated by javadoc. The JDK docs build doesn't generate javadoc for the jdk.unsupported module so maybe it doesn't matter but here's a suggestion for something a bit briefer:

"The sensitivity levels were historically used by polling based WatchService implementations to configure the polling interval. They are are no longer used. The WatchService implementations in the JDK ignore these WatchEvent modifiers if they are specified when registering a directory to be watched."

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

PR: https://git.openjdk.org/jdk/pull/12746