RFR: JDK-8303912 - Clean up JavadocTokenizer [v2]
    Jonathan Gibbons 
    jjg at openjdk.org
       
    Mon Mar 27 16:55:21 UTC 2023
    
    
  
On Fri, 17 Mar 2023 16:27:10 GMT, Jim Laskey <jlaskey at openjdk.org> wrote:
>> Previous work had cleaned up the JavacTokenizer, but left JavadocTokenizer alone, mostly due to its perplexing state. This is follow up work to simplify the JavadocTokenizer.
>
> Jim Laskey has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - Corrections from review
>  - Merge branch 'master' into 8303912
>  - Clean up JavadocTokenizer
src/jdk.compiler/share/classes/com/sun/tools/javac/parser/JavaTokenizer.java line 1200:
> 1198:          * LINE starting with //
> 1199:          * BLOCK starting with /*
> 1200:          * JAVADOC starting with /**
The detail here would be better as comments on the enum members.
src/jdk.compiler/share/classes/com/sun/tools/javac/parser/JavaTokenizer.java line 1250:
> 1248:          * LINE starting with //
> 1249:          * BLOCK starting with /*
> 1250:          * JAVADOC starting with /**
duplicates info on the decl for `cs`; suggest moving the info to the enum declaration members
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/12953#discussion_r1149541782
PR Review Comment: https://git.openjdk.org/jdk/pull/12953#discussion_r1149544294
    
    
More information about the compiler-dev
mailing list