Minimize SpawnAction constructor exposure

One's unused, the other is ok as protected.

--
PiperOrigin-RevId: 145291062
MOS_MIGRATED_REVID=145291062

1 files changed, 3 insertions, 10 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/actions/SpawnAction.java b/src/main/java/com/google/devtools/build/lib/analysis/actions/SpawnAction.java
index 607028f5cb..b7a094a469 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/actions/SpawnAction.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/actions/SpawnAction.java
@@ -437,11 +437,11 @@ public class SpawnAction extends AbstractAction implements ExecutionInfoSpecifie
 
     /**
      * Creates an ActionSpawn with the given environment variables.
-     * 
+     *
      * <p>Subclasses of ActionSpawn may subclass in order to provide action-specific values for
      * environment variables or action inputs.
      */
-    public ActionSpawn(Map<String, String> clientEnv) {
+    protected ActionSpawn(Map<String, String> clientEnv) {
       super(ImmutableList.copyOf(argv.arguments()),
           ImmutableMap.<String, String>of(),
           executionInfo,
@@ -467,13 +467,6 @@ public class SpawnAction extends AbstractAction implements ExecutionInfoSpecifie
       effectiveEnvironment = ImmutableMap.copyOf(env);
     }
 
-    /**
-     * Creates an ActionSpawn with no environment variables.
-     */
-    public ActionSpawn() {
-      this(null);
-    }
-
     @Override
     public ImmutableMap<String, String> getEnvironment() {
       return effectiveEnvironment;
@@ -1132,7 +1125,7 @@ public class SpawnAction extends AbstractAction implements ExecutionInfoSpecifie
 
     public <T> Builder setExtraActionInfo(
         GeneratedExtension<ExtraActionInfo, T> extension, T value) {
-      this.extraActionInfoSupplier = new ExtraActionInfoSupplier<T>(extension, value);
+      this.extraActionInfoSupplier = new ExtraActionInfoSupplier<>(extension, value);
       return this;
     }