RFR: JDK-8297412: Remove easy warnings in javafx.fxml, javafx.media, javafx.swing, javafx.swt and javafx.web [v4]

Kevin Rushforth kcr at openjdk.org
Tue Nov 22 19:16:04 UTC 2022


On Tue, 22 Nov 2022 19:06:47 GMT, John Hendrikx <jhendrikx at openjdk.org> wrote:

>> - 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:
> 
>   Fix indent of 5 spaces to 4 spaces

Given that some changes will require a bit more review, I'd like two reviewers.

-------------

PR: https://git.openjdk.org/jfx/pull/958


More information about the openjfx-dev mailing list