[prone] Apply UnusedMethod

Author: Vincent Potucek

gradle/error-prone.gradle

@@ -23,11 +23,6 @@ tasks.withType(JavaCompile).configureEach {
 				'NonStaticImport',
 				'Slf4jLoggerDeclaration', // logger -> log
 				'StaticImport',
-				// critical, finds lots of bugs... unused configs and stuff.
-				'Unused',
-				'UnusedMethod',
-				'UnusedParameters',
-				'UnusedVariable',
 				)
 		error(
 				'AmbiguousJsonCreator',
@@ -52,11 +47,15 @@ tasks.withType(JavaCompile).configureEach {
 				'SelfAssignment',
 				'StringJoin',
 				'StringJoining',
+				'Unused',
 				)
 		// fixme bug: this only happens when the file is dirty!
 		// up2date checking (caching) must consider file changes, as file is currently corrupt!
 		// fix SelfTest.java:L22 ForbidGradleInternal(import org.gradle.api.internal.project.ProjectInternal;)
 		errorproneArgs.add('-XepExcludedPaths:.*/SelfTest.java')
+		// bug @MethodSource
+		errorproneArgs.add('-XepExcludedPaths:.*/EclipseJdtFormatterStepTest.java')
+		errorproneArgs.add('-XepExcludedPaths:.*/JvmTest.java')
 		if (!getenv().containsKey('CI') && getenv('IN_PLACE')?.toBoolean()) {
 			errorproneArgs.addAll(
 					'-XepPatchLocation:IN_PLACE',
@@ -86,7 +85,10 @@ tasks.withType(JavaCompile).configureEach {
 					'Slf4jLogStatement,' +
 					'StringJoin,' +
 					'StringJoining,' +
-					'TimeZoneUsage,'
+					'TimeZoneUsage,' +
+					'UnusedMethod,' +
+					'UnusedParameters,' +
+					'UnusedVariable,'
 					)
 		}
 	}

lib/src/compatKtLint0Dot49Dot0/java/com/diffplug/spotless/glue/ktlint/compat/KtLintCompat0Dot49Dot0Adapter.java

@@ -27,9 +27,6 @@
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import com.pinterest.ktlint.cli.ruleset.core.api.RuleSetProviderV3;
 import com.pinterest.ktlint.rule.engine.api.Code;
 import com.pinterest.ktlint.rule.engine.api.EditorConfigDefaults;
@@ -55,8 +52,6 @@
 
 public class KtLintCompat0Dot49Dot0Adapter implements KtLintCompatAdapter {
 
-	private static final Logger LOGGER = LoggerFactory.getLogger(KtLintCompat0Dot49Dot0Adapter.class);
-
 	private static final List<EditorConfigProperty<?>> DEFAULT_EDITOR_CONFIG_PROPERTIES;
 
 	static {

lib/src/compatKtLint0Dot50Dot0/java/com/diffplug/spotless/glue/ktlint/compat/KtLintCompat0Dot50Dot0Adapter.java

@@ -25,9 +25,6 @@
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import com.pinterest.ktlint.cli.ruleset.core.api.RuleSetProviderV3;
 import com.pinterest.ktlint.rule.engine.api.Code;
 import com.pinterest.ktlint.rule.engine.api.EditorConfigDefaults;
@@ -55,8 +52,6 @@
 
 public class KtLintCompat0Dot50Dot0Adapter implements KtLintCompatAdapter {
 
-	private static final Logger LOGGER = LoggerFactory.getLogger(KtLintCompat0Dot50Dot0Adapter.class);
-
 	private static final List<EditorConfigProperty<?>> DEFAULT_EDITOR_CONFIG_PROPERTIES;
 
 	static {

lib/src/compatKtLint1Dot0Dot0/java/com/diffplug/spotless/glue/ktlint/compat/KtLintCompat1Dot0Dot0Adapter.java

@@ -24,9 +24,6 @@
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import com.pinterest.ktlint.cli.ruleset.core.api.RuleSetProviderV3;
 import com.pinterest.ktlint.rule.engine.api.Code;
 import com.pinterest.ktlint.rule.engine.api.EditorConfigDefaults;
@@ -54,8 +51,6 @@
 
 public class KtLintCompat1Dot0Dot0Adapter implements KtLintCompatAdapter {
 
-	private static final Logger LOGGER = LoggerFactory.getLogger(KtLintCompat1Dot0Dot0Adapter.class);
-
 	private static final List<EditorConfigProperty<?>> DEFAULT_EDITOR_CONFIG_PROPERTIES;
 
 	static {

lib/src/gherkin/java/com/diffplug/spotless/glue/gherkin/GherkinUtilsFormatterFunc.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2023 DiffPlug
+ * Copyright 2023-2025 DiffPlug
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -15,9 +15,6 @@
  */
 package com.diffplug.spotless.glue.gherkin;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import com.diffplug.spotless.FormatterFunc;
 import com.diffplug.spotless.gherkin.GherkinUtilsConfig;
 
@@ -30,12 +27,9 @@
 import io.cucumber.messages.types.SourceMediaType;
 
 public class GherkinUtilsFormatterFunc implements FormatterFunc {
-	private static final Logger LOGGER = LoggerFactory.getLogger(GherkinUtilsFormatterFunc.class);
-
-	private final GherkinUtilsConfig gherkinSimpleConfig;
 
 	public GherkinUtilsFormatterFunc(GherkinUtilsConfig gherkinSimpleConfig) {
-		this.gherkinSimpleConfig = gherkinSimpleConfig;
+
 	}
 
 	// Follows https://github.com/cucumber/gherkin-utils/blob/main/java/src/test/java/io/cucumber/gherkin/utils/pretty/PrettyTest.java

lib/src/googleJavaFormat/java/com/diffplug/spotless/glue/java/GoogleJavaFormatRemoveUnusedImporterFormatterFunc.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2023 DiffPlug
+ * Copyright 2023-2025 DiffPlug
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -15,8 +15,6 @@
  */
 package com.diffplug.spotless.glue.java;
 
-import java.util.Objects;
-
 import javax.annotation.Nonnull;
 
 import com.google.googlejavaformat.java.RemoveUnusedImports;
@@ -25,11 +23,8 @@
 
 public class GoogleJavaFormatRemoveUnusedImporterFormatterFunc implements FormatterFunc {
 
-	@Nonnull
-	private final String version;
-
 	public GoogleJavaFormatRemoveUnusedImporterFormatterFunc(@Nonnull String version) {
-		this.version = Objects.requireNonNull(version);
+
 	}
 
 	@Override

lib/src/main/java/com/diffplug/spotless/generic/ReplaceRegexStep.java

@@ -68,12 +68,12 @@ private static final class LintState implements Serializable {
 		private static final long serialVersionUID = 1L;
 
 		private final Pattern regex;
-		private final String ruleId;
+
 		private final String lintDetail;
 
 		LintState(Pattern regex, String ruleId, String lintDetail) {
 			this.regex = regex;
-			this.ruleId = ruleId;
+
 			this.lintDetail = lintDetail;
 		}
 

lib/src/main/java/com/diffplug/spotless/java/PalantirJavaFormatStep.java

@@ -111,14 +111,14 @@ private static final class State implements Serializable {
 		private static final long serialVersionUID = 1L;
 
 		private final JarState jarState;
-		private final String formatterVersion;
+
 		private final String style;
 		private final boolean formatJavadoc;
 
 		State(JarState jarState, String formatterVersion, String style, boolean formatJavadoc) {
 			ModuleHelper.doOpenInternalPackagesIfRequired();
 			this.jarState = jarState;
-			this.formatterVersion = formatterVersion;
+
 			this.style = style;
 			this.formatJavadoc = formatJavadoc;
 		}

lib/src/main/java/com/diffplug/spotless/npm/NpmFormatterStepStateBase.java

@@ -41,18 +41,15 @@ abstract class NpmFormatterStepStateBase implements Serializable {
 
 	private static final Logger LOGGER = LoggerFactory.getLogger(NpmFormatterStepStateBase.class);
 
-	private static final TimedLogger TIMED_LOGGER = TimedLogger.forLogger(LOGGER);
-
 	@Serial
 	private static final long serialVersionUID = 1460749955865959948L;
 
-	private final String stepName;
 	private final NpmConfig npmConfig;
 
 	public final NpmFormatterStepLocations locations;
 
 	protected NpmFormatterStepStateBase(String stepName, NpmConfig npmConfig, NpmFormatterStepLocations locations) throws IOException {
-		this.stepName = requireNonNull(stepName);
+
 		this.npmConfig = requireNonNull(npmConfig);
 		this.locations = locations;
 	}

lib/src/main/java/com/diffplug/spotless/rdf/RdfFormatterFunc.java

@@ -64,13 +64,13 @@ public String apply(String rawUnix, File file) throws Exception {
 				return formatTurtle(rawUnix, file, reflectionHelper);
 			}
 			if (TRIG_EXTENSIONS.contains(extension)) {
-				return formatTrig(rawUnix, file);
+				return formatTrig(file);
 			}
 			if (NTRIPLES_EXTENSIONS.contains(extension)) {
-				return formatNTriples(rawUnix, file);
+				return formatNTriples(file);
 			}
 			if (NQUADS_EXTENSIONS.contains(extension)) {
-				return formatNQuads(rawUnix, file);
+				return formatNQuads(rawUnix);
 			}
 			throw new IllegalArgumentException("Cannot handle file with extension %s".formatted(extension));
 		} catch (InvocationTargetException e) {
@@ -80,15 +80,15 @@ public String apply(String rawUnix, File file) throws Exception {
 		}
 	}
 
-	private String formatNQuads(String rawUnix, File file) {
+	private String formatNQuads(String rawUnix) {
 		throw new UnsupportedOperationException("NQUADS formatting not supported yet");
 	}
 
-	private String formatNTriples(String rawUnix, File file) {
+	private String formatNTriples(File file) {
 		throw new UnsupportedOperationException("NTRIPLES formatting not supported yet");
 	}
 
-	private String formatTrig(String rawUnix, File file) {
+	private String formatTrig(File file) {
 		throw new UnsupportedOperationException("TRIG formatting not supported yet");
 	}