Automatic code cleanup.
PiperOrigin-RevId: 458423766
Change-Id: I0c3ee34d8e766bb654b8484d9ff71892baf76847
diff --git a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/WerrorCustomOption.java b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/WerrorCustomOption.java
index 3249f36..a74b9f6 100644
--- a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/WerrorCustomOption.java
+++ b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/WerrorCustomOption.java
@@ -19,6 +19,7 @@
import com.google.common.base.Splitter;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableMap;
+import com.google.errorprone.annotations.CanIgnoreReturnValue;
import java.util.LinkedHashMap;
import java.util.Map;
@@ -63,12 +64,14 @@
}
}
+ @CanIgnoreReturnValue
Builder all() {
werrors.clear();
werrors.put("all", true);
return this;
}
+ @CanIgnoreReturnValue
Builder process(String flag) {
checkArgument(flag.startsWith(WERROR), flag);
for (String arg : Splitter.on(',').split(flag.substring(WERROR.length()))) {
diff --git a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/plugins/dependency/DependencyModule.java b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/plugins/dependency/DependencyModule.java
index d6917e9..7e73370 100644
--- a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/plugins/dependency/DependencyModule.java
+++ b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/plugins/dependency/DependencyModule.java
@@ -27,6 +27,7 @@
import com.google.devtools.build.lib.view.proto.Deps.Dependencies;
import com.google.devtools.build.lib.view.proto.Deps.Dependency;
import com.google.devtools.build.lib.view.proto.Deps.Dependency.Kind;
+import com.google.errorprone.annotations.CanIgnoreReturnValue;
import com.sun.tools.javac.code.Symbol.PackageSymbol;
import java.io.BufferedInputStream;
import java.io.BufferedOutputStream;
@@ -384,6 +385,7 @@
* @param strictJavaDeps level, as specified by {@link StrictJavaDeps}
* @return this Builder instance
*/
+ @CanIgnoreReturnValue
public Builder setStrictJavaDeps(String strictJavaDeps) {
this.strictJavaDeps = StrictJavaDeps.valueOf(strictJavaDeps);
return this;
@@ -395,6 +397,7 @@
* @param fixDepsTool tool name
* @return this Builder instance
*/
+ @CanIgnoreReturnValue
public Builder setFixDepsTool(FixTool fixDepsTool) {
this.fixDepsTool = fixDepsTool;
return this;
@@ -406,12 +409,14 @@
* @param targetLabel label, such as the label of a RuleConfiguredTarget.
* @return this Builder instance.
*/
+ @CanIgnoreReturnValue
public Builder setTargetLabel(String targetLabel) {
this.targetLabel = targetLabel;
return this;
}
/** Sets the paths to jars that are direct dependencies. */
+ @CanIgnoreReturnValue
public Builder setDirectJars(ImmutableSet<Path> directJars) {
this.directJars = directJars;
return this;
@@ -424,6 +429,7 @@
* @param outputDepsProtoFile output file name for dependency information
* @return this Builder instance
*/
+ @CanIgnoreReturnValue
public Builder setOutputDepsProtoFile(Path outputDepsProtoFile) {
this.outputDepsProtoFile = outputDepsProtoFile;
return this;
@@ -435,12 +441,14 @@
* @param depsArtifacts dependency artifacts
* @return this Builder instance
*/
+ @CanIgnoreReturnValue
public Builder addDepsArtifacts(Collection<Path> depsArtifacts) {
this.depsArtifacts.addAll(depsArtifacts);
return this;
}
/** Sets the platform classpath entries. */
+ @CanIgnoreReturnValue
public Builder setPlatformJars(ImmutableSet<Path> platformJars) {
this.platformJars = platformJars;
return this;
@@ -451,6 +459,7 @@
*
* @return this Builder instance
*/
+ @CanIgnoreReturnValue
public Builder setReduceClasspath() {
this.strictClasspathMode = true;
return this;
@@ -462,6 +471,7 @@
* @param fixMessage the fix message
* @return this Builder instance
*/
+ @CanIgnoreReturnValue
public Builder setFixMessage(FixMessage fixMessage) {
this.fixMessage = fixMessage;
return this;
@@ -473,6 +483,7 @@
* @param exemptGenerator the generator class name
* @return this Builder instance
*/
+ @CanIgnoreReturnValue
public Builder addExemptGenerator(String exemptGenerator) {
exemptGenerators.add(exemptGenerator);
return this;
diff --git a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/statistics/BlazeJavacStatistics.java b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/statistics/BlazeJavacStatistics.java
index ddb9218..06d3bf3 100644
--- a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/statistics/BlazeJavacStatistics.java
+++ b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/statistics/BlazeJavacStatistics.java
@@ -18,6 +18,7 @@
import com.google.auto.value.AutoValue;
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.ImmutableSet;
+import com.google.errorprone.annotations.CanIgnoreReturnValue;
import com.sun.tools.javac.util.Context;
import java.time.Duration;
import java.util.Optional;
@@ -113,6 +114,7 @@
public abstract Builder transitiveClasspathFallback(boolean fallback);
+ @CanIgnoreReturnValue
public Builder addBugpatternTiming(String key, Duration value) {
bugpatternTimingBuilder().put(key, value);
return this;
@@ -129,11 +131,13 @@
* the byte[] data type. If we find a way to make this more safe, we would prefer to use a
* protobuf ByteString instead for its immutability.
*/
+ @CanIgnoreReturnValue
public Builder addAuxiliaryData(AuxiliaryDataSource key, byte[] serializedData) {
auxiliaryDataBuilder().put(key, serializedData.clone());
return this;
}
+ @CanIgnoreReturnValue
public Builder addProcessor(String processor) {
processorsBuilder().add(processor);
return this;
