RFR: JDK-8297412: Remove easy warnings in javafx.fxml, javafx.media, javafx.swing, javafx.swt and javafx.web [v5]
John Hendrikx
jhendrikx at openjdk.org
Mon Nov 28 17:04:17 UTC 2022
> - Remove unsupported/unnecessary SuppressWarning annotations
> - Remove reduntant type specifications (use diamond operator)
> - Remove unused or duplicate imports
> - Remove unnecessary casts (type is already correct type or can be autoboxed)
> - Remove unnecessary semi-colons (at end of class definitions, or just repeated ones)
> - Remove redundant super interfaces (interface that is already inherited)
> - Remove unused type parameters
> - Remove declared checked exceptions that are never thrown
> - Add missing `@Override` annotations
John Hendrikx has updated the pull request incrementally with one additional commit since the last revision:
Revert change of private class to abstract in FXMLLoader
Exception is now documented instead
-------------
Changes:
- all: https://git.openjdk.org/jfx/pull/958/files
- new: https://git.openjdk.org/jfx/pull/958/files/a5bbff82..91e01c26
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jfx&pr=958&range=04
- incr: https://webrevs.openjdk.org/?repo=jfx&pr=958&range=03-04
Stats: 22 lines in 1 file changed: 11 ins; 10 del; 1 mod
Patch: https://git.openjdk.org/jfx/pull/958.diff
Fetch: git fetch https://git.openjdk.org/jfx pull/958/head:pull/958
PR: https://git.openjdk.org/jfx/pull/958
More information about the openjfx-dev
mailing list