RFR: JDK-8286153: Remove redundant casts and other cleanup [v2]
Jonathan Gibbons
jjg at openjdk.java.net
Thu May 5 17:20:25 UTC 2022
> Please review some cleanup updates to address issues reported by an IDE.
>
> The seeds for the change were a series of redundant casts, that have now all been removed. Various other warnings and suggestions were made by the IDE for the affected files. There were a number of places with redundant type arguments, for which the general fix was in favor of using `var` instead of `<>`.
>
> Some `switch` statements were converted to the enhanced `switch` form, which also revealed a couple of places where `RECORD` should have been added alongside `ENUM`.
Jonathan Gibbons has updated the pull request incrementally with one additional commit since the last revision:
address review feedback
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/8543/files
- new: https://git.openjdk.java.net/jdk/pull/8543/files/e18b0b6c..6ad78546
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=8543&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=8543&range=00-01
Stats: 25 lines in 3 files changed: 3 ins; 7 del; 15 mod
Patch: https://git.openjdk.java.net/jdk/pull/8543.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/8543/head:pull/8543
PR: https://git.openjdk.java.net/jdk/pull/8543
More information about the javadoc-dev
mailing list