RFR: 8307409: Refactor usage examples to use @snippet in the java.nio packages
    Roger Riggs 
    rriggs at openjdk.org
       
    Mon May  8 20:58:29 UTC 2023
    
    
  
On Mon, 8 May 2023 19:07:25 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:
> Convert the `<pre>{@code ...}</pre>` and some other patterns to `{@snippet ..}`.
src/java.base/share/classes/java/nio/X-Buffer.java.template line 2244:
> 2242:      * {@snippet lang=java :
> 2243:      * nb.alignmentOffset(0, unitSize) == 0
> 2244:      * nb.alignmentOffset(nb.limit(), unitSize) == 0
The indentation (within the files) is inconsistent, I think at 4 spaces might be an improvement.
src/java.base/share/classes/java/nio/channels/SelectionKey.java line 343:
> 341:      * <blockquote><pre>{@code
> 342:      * k.readyOps() & OP_READ != 0
> 343:      * }</pre></blockquote>
The blockquote provided some indentation, that might make the example more readable.
src/java.base/share/classes/java/nio/file/Path.java line 849:
> 847:      * invocation
> 848:      * {@snippet lang=java :
> 849:      *     watchable.{@link #register(WatchService,WatchEvent.Kind[],WatchEvent.Modifier[]) register}(watcher, events, new WatchEvent.Modifier[0]);
Take out the {@link, it isn't needed and gets included literally.  (And make the line very long).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13870#discussion_r1187885244
PR Review Comment: https://git.openjdk.org/jdk/pull/13870#discussion_r1187879709
PR Review Comment: https://git.openjdk.org/jdk/pull/13870#discussion_r1187890625
    
    
More information about the nio-dev
mailing list