On Tue, 29 Nov 2022 07:41:24 GMT, John Hendrikx <jhendr...@openjdk.org> wrote:
>> Note: I ran into a `javac` compiler bug while replacing types with diamond >> operators (ecj has no issues). I had two options, add a >> `SuppressWarnings("unused")` or to use a lambda instead of a method >> reference to make `javac` happy. I choose the later and added a comment so >> it can be fixed once the bug is fixed. I've reported the issue here: >> https://bugs.openjdk.org/browse/JDK-8297428 >> >> - 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 some indents and put declarations on one line where possible oh no, my PRs are going to wreak havoc with your large PRs. let's make sure yours get in first. ------------- PR: https://git.openjdk.org/jfx/pull/959