Java 8 follow-up changes #2747

Note that for the `final` variable warnings, it seems some of those variables were made `final` intentionally, without the compiler requiring it. But for consistency I removed the `final` from them as well.

@OverRide

Is disabled by default as well. It seems what this pattern does is suggest to replace anonymous classes stored in constants with a corresponding method (with the intention that the developer then uses a method reference to that method). But it even does this when the anonymous class implements a custom functional type, and not a generic type such as `Function`. For example it suggested to replace ``` private static final FieldNamingStrategy CUSTOM_FIELD_NAMING_STRATEGY = new FieldNamingStrategy() { @OverRide public String translateName(Field f) { return "foo"; } }; ``` with a method ``` private static String customFieldNamingStrategy(Field f) { return "foo"; } ``` Which is probably not that helpful, and might not actually increase readability.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Java 8 follow-up changes #2747

Java 8 follow-up changes #2747

Commits on Sep 23, 2024