> 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: Revert instanceof changes and replace with null checks ------------- Changes: - all: https://git.openjdk.org/jfx/pull/959/files - new: https://git.openjdk.org/jfx/pull/959/files/f56b8aae..1341a438 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=959&range=03 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=959&range=02-03 Stats: 6 lines in 1 file changed: 2 ins; 0 del; 4 mod Patch: https://git.openjdk.org/jfx/pull/959.diff Fetch: git fetch https://git.openjdk.org/jfx pull/959/head:pull/959 PR: https://git.openjdk.org/jfx/pull/959