RFR: 8070: Move to JDK 17 leftovers

Brice Dutheil bdutheil at openjdk.org
Wed May 17 16:03:37 UTC 2023


On Wed, 17 May 2023 15:04:09 GMT, Brice Dutheil <bdutheil at openjdk.org> wrote:

> In #482 I forgot to update the Docker build, as well as remove some old profiles in the parent pom. This PR, fixes that. Also I noticed a few quirks around ECJ compiler that I think should be removed as they are not working when using a regular OpenJDK compiler.

application/org.openjdk.jmc.ui/src/main/java/org/openjdk/jmc/ui/misc/DndToolkit.java line 120:

> 118: 		IDropAction<List<U>, T> action,
> 119: 		IDropValidator<List<U>, T> validator
> 120: 	) {

**question:** It seems this change is not accepted by the formatter, I can change it back, but do you think we could improve readability if the formatter rules were updated.


Error:  Failed to execute goal com.diffplug.spotless:spotless-maven-plugin:2.17.2:check (default-cli) on project org.openjdk.jmc.ui: The following files had format violations:
Error:      src/main/java/org/openjdk/jmc/ui/misc/DndToolkit.java
Error:          @@ -112,12 +112,8 @@
Error:           \t}
Error:           
Error:           \tpublic·static·<T,·U>·ViewerDropAdapter·createLocalDropListTarget(
Error:          -\t\tViewer·viewer,
Error:          -\t\tClass<T>·targetType,
Error:          -\t\tClass<U>·srcType,
Error:          -\t\tIDropAction<List<U>,·T>·action,
Error:          -\t\tIDropValidator<List<U>,·T>·validator
Error:          -\t)·{
Error:          +\t\tViewer·viewer,·Class<T>·targetType,·Class<U>·srcType,·IDropAction<List<U>,·T>·action,
Error:          +\t\tIDropValidator<List<U>,·T>·validator)·{
Error:           \t\treturn·createLocalDropTarget(viewer,·targetType,·new·IDropAction<ISelection,·T>()·{
Error:           
Error:           \t\t\t at SuppressWarnings("unchecked")
Error:  Run 'mvn spotless:apply' to fix these violations.
Error:  -> [Help 1]

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

PR Review Comment: https://git.openjdk.org/jmc/pull/483#discussion_r1196718922


More information about the jmc-dev mailing list