Minimize SpawnAction constructor exposure
One's unused, the other is ok as protected.
MOS_MIGRATED_REVID=145291062
@@ -441,7 +441,7 @@ public class SpawnAction extends AbstractAction implements ExecutionInfoSpecifie
* <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;
}
