Spotbugs: unnecessary boxing.

nedtwigg · nedtwigg · commit 0e734bd04612 · 2021-11-08T21:37:15.000-08:00
diff --git a/lib/src/main/java/com/diffplug/spotless/Jvm.java b/lib/src/main/java/com/diffplug/spotless/Jvm.java
@@ -242,7 +242,7 @@ public int compare(V version0, V version1) {
 
 			private static <V> int[] convert(V versionObject) {
 				try {
-					return Arrays.asList(versionObject.toString().split("\\.")).stream().mapToInt(s -> Integer.valueOf(s)).toArray();
+					return Arrays.asList(versionObject.toString().split("\\.")).stream().mapToInt(Integer::parseInt).toArray();
 				} catch (Exception e) {
 					throw new IllegalArgumentException(String.format("Not a semantic version: %s", versionObject), e);
 				}

Original file line number	Diff line number	Diff line change
`@@ -242,7 +242,7 @@ public int compare(V version0, V version1) {`
`242`	`242`
`243`	`243`	`private static <V> int[] convert(V versionObject) {`
`244`	`244`	`try {`
`245`		`- return Arrays.asList(versionObject.toString().split("\\.")).stream().mapToInt(s -> Integer.valueOf(s)).toArray();`
	`245`	`+ return Arrays.asList(versionObject.toString().split("\\.")).stream().mapToInt(Integer::parseInt).toArray();`
`246`	`246`	`} catch (Exception e) {`
`247`	`247`	`throw new IllegalArgumentException(String.format("Not a semantic version: %s", versionObject), e);`
`248`	`248`	`}`