[commands] Suppress this-escape warnings (wpilibsuite#6828)

KangarooKoala · Jul 13, 2024 · fc44737 · fc44737
1 parent deb5f3d
commit fc44737
Show file tree

Hide file tree

Showing 8 changed files with 9 additions and 0 deletions.
diff --git a/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ConditionalCommand.java b/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ConditionalCommand.java
@@ -32,6 +32,7 @@ public class ConditionalCommand extends Command {
    * @param onFalse the command to run if the condition is false
    * @param condition the condition to determine which command to run
    */
+  @SuppressWarnings("this-escape")
   public ConditionalCommand(Command onTrue, Command onFalse, BooleanSupplier condition) {
     m_onTrue = requireNonNullParam(onTrue, "onTrue", "ConditionalCommand");
     m_onFalse = requireNonNullParam(onFalse, "onFalse", "ConditionalCommand");

diff --git a/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/PIDCommand.java b/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/PIDCommand.java
@@ -39,6 +39,7 @@ public class PIDCommand extends Command {
    * @param useOutput the controller's output
    * @param requirements the subsystems required by this command
    */
+  @SuppressWarnings("this-escape")
   public PIDCommand(
       PIDController controller,
       DoubleSupplier measurementSource,

diff --git a/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ParallelCommandGroup.java b/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ParallelCommandGroup.java
@@ -32,6 +32,7 @@ public class ParallelCommandGroup extends Command {
    *
    * @param commands the commands to include in this composition.
    */
+  @SuppressWarnings("this-escape")
   public ParallelCommandGroup(Command... commands) {
     addCommands(commands);
   }

diff --git a/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ParallelDeadlineGroup.java b/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ParallelDeadlineGroup.java
@@ -40,6 +40,7 @@ public class ParallelDeadlineGroup extends Command {
    * @param otherCommands the other commands to be executed
    * @throws IllegalArgumentException if the deadline command is also in the otherCommands argument
    */
+  @SuppressWarnings("this-escape")
   public ParallelDeadlineGroup(Command deadline, Command... otherCommands) {
     setDeadline(deadline);
     addCommands(otherCommands);

diff --git a/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ParallelRaceGroup.java b/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ParallelRaceGroup.java
@@ -32,6 +32,7 @@ public class ParallelRaceGroup extends Command {
    *
    * @param commands the commands to include in this composition.
    */
+  @SuppressWarnings("this-escape")
   public ParallelRaceGroup(Command... commands) {
     addCommands(commands);
   }

diff --git a/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ProfiledPIDCommand.java b/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/ProfiledPIDCommand.java
@@ -42,6 +42,7 @@ public class ProfiledPIDCommand extends Command {
    * @param useOutput the controller's output
    * @param requirements the subsystems required by this command
    */
+  @SuppressWarnings("this-escape")
   public ProfiledPIDCommand(
       ProfiledPIDController controller,
       DoubleSupplier measurementSource,
@@ -70,6 +71,7 @@ public ProfiledPIDCommand(
    * @param useOutput the controller's output
    * @param requirements the subsystems required by this command
    */
+  @SuppressWarnings("this-escape")
   public ProfiledPIDCommand(
       ProfiledPIDController controller,
       DoubleSupplier measurementSource,

diff --git a/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/SelectCommand.java b/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/SelectCommand.java
@@ -38,6 +38,7 @@ public class SelectCommand<K> extends Command {
    * @param commands the map of commands to choose from
    * @param selector the selector to determine which command to run
    */
+  @SuppressWarnings("this-escape")
   public SelectCommand(Map<K, Command> commands, Supplier<? extends K> selector) {
     m_commands = requireNonNullParam(commands, "commands", "SelectCommand");
     m_selector = requireNonNullParam(selector, "selector", "SelectCommand");

diff --git a/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/SequentialCommandGroup.java b/wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/SequentialCommandGroup.java
@@ -29,6 +29,7 @@ public class SequentialCommandGroup extends Command {
    *
    * @param commands the commands to include in this composition.
    */
+  @SuppressWarnings("this-escape")
   public SequentialCommandGroup(Command... commands) {
     addCommands(commands);
   }