RFR: JDK-8305591: Cleanup use of `newline` flag in DocCommentParser
Jonathan Gibbons
jjg at openjdk.org
Tue Apr 4 21:03:21 UTC 2023
Please remove a simple cleanup fix, to remove unnecessary assignments to set the `newline` flag to `true` in `DocCommentParser`. The flag is always set appropriately in `nextChar()`.
This removes a number of "fall-through" cases in switch statements. In reviewing the use of `@SuppressWarnings("fall through")` it was noted that there is a missing `break` in `case '@'` in `inlineWord()`. Fixing this breaks a test. This will be addressed separately. [JDK-8305620](https://bugs.openjdk.org/browse/JDK-8305620)
-------------
Commit messages:
- JDK-8305591: Cleanup use of `newline` flag in DocCommentParser
Changes: https://git.openjdk.org/jdk/pull/13341/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=13341&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8305591
Stats: 18 lines in 1 file changed: 0 ins; 18 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/13341.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/13341/head:pull/13341
PR: https://git.openjdk.org/jdk/pull/13341
More information about the compiler-dev
mailing list