commit 4192ca611f272c71611c21187d8b2c4b2bde71ff
author Kurt Alfred Kluever <kak@google.com> Tue Jul 05 06:30:29 2022 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Jul 05 06:32:21 2022 -0700
tree 8a72213ea5bb937bcbb12d9675c9c0934d01c0b2
parent 1b50b2cf539c0dbaa8a2b00913e3fdc9ccf915b4

Automatic code cleanup.

PiperOrigin-RevId: 459067450
Change-Id: Ib4fec6f79f98fb5ed2f20083539a02ed6159ecbc

src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java b/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java
index 478b191..58dc511 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java
@@ -97,6 +97,7 @@
 import com.google.devtools.build.lib.util.StringUtil;
 import com.google.devtools.build.lib.vfs.FileSystemUtils;
 import com.google.devtools.build.lib.vfs.PathFragment;
+import com.google.errorprone.annotations.CanIgnoreReturnValue;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.HashMap;
@@ -1736,31 +1737,37 @@
       }
     }
 
+    @CanIgnoreReturnValue
     public Builder setRuleClassProvider(ConfiguredRuleClassProvider ruleClassProvider) {
       this.ruleClassProvider = ruleClassProvider;
       return this;
     }
 
+    @CanIgnoreReturnValue
     public Builder setHostConfiguration(BuildConfigurationValue hostConfiguration) {
       this.hostConfiguration = hostConfiguration;
       return this;
     }
 
+    @CanIgnoreReturnValue
     public Builder setConfigurationFragmentPolicy(ConfigurationFragmentPolicy policy) {
       this.configurationFragmentPolicy = policy;
       return this;
     }
 
+    @CanIgnoreReturnValue
     public Builder setActionOwnerSymbol(ActionLookupKey actionOwnerSymbol) {
       this.actionOwnerSymbol = actionOwnerSymbol;
       return this;
     }
 
+    @CanIgnoreReturnValue
     public Builder setMutability(Mutability mutability) {
       this.mutability = mutability;
       return this;
     }
 
+    @CanIgnoreReturnValue
     public Builder setVisibility(NestedSet<PackageGroupContents> visibility) {
       this.visibility = visibility;
       return this;
@@ -1770,6 +1777,7 @@
      * Sets the prerequisites and checks their visibility. It also generates appropriate error or
      * warning messages and sets the error flag as appropriate.
      */
+    @CanIgnoreReturnValue
     public Builder setPrerequisites(
         OrderedSetMultimap<Attribute, ConfiguredTargetAndData> prerequisiteMap) {
       this.prerequisiteMap = Preconditions.checkNotNull(prerequisiteMap);
@@ -1777,6 +1785,7 @@
     }
 
     /** Adds attributes which are defined by an Aspect (and not by RuleClass). */
+    @CanIgnoreReturnValue
     public Builder setAspectAttributes(Map<String, Attribute> aspectAttributes) {
       this.aspectAttributes = ImmutableMap.copyOf(aspectAttributes);
       return this;
@@ -1786,12 +1795,14 @@
      * Sets the configuration conditions needed to determine which paths to follow for this rule's
      * configurable attributes.
      */
+    @CanIgnoreReturnValue
     public Builder setConfigConditions(ConfigConditions configConditions) {
       this.configConditions = Preconditions.checkNotNull(configConditions);
       return this;
     }
 
     /** Sets the collection of {@link ResolvedToolchainContext}s available to this rule. */
+    @CanIgnoreReturnValue
     @VisibleForTesting
     public Builder setToolchainContexts(
         ToolchainCollection<ResolvedToolchainContext> toolchainContexts) {
@@ -1802,6 +1813,7 @@
       return this;
     }
 
+    @CanIgnoreReturnValue
     public Builder setExecGroupCollectionBuilder(
         ExecGroupCollection.Builder execGroupCollectionBuilder) {
       this.execGroupCollectionBuilder = execGroupCollectionBuilder;
@@ -1812,11 +1824,13 @@
      * Warning: if you set the exec properties using this method any exec_properties attribute value
      * will be ignored in favor of this value.
      */
+    @CanIgnoreReturnValue
     public Builder setExecProperties(ImmutableMap<String, String> execProperties) {
       this.rawExecProperties = execProperties;
       return this;
     }
 
+    @CanIgnoreReturnValue
     public Builder setRequiredConfigFragments(
         @Nullable RequiredConfigFragmentsProvider requiredConfigFragments) {
       this.requiredConfigFragments = requiredConfigFragments;