Better comments and variable names

Author: kytpbs

wpilibNewCommands/src/main/java/edu/wpi/first/wpilibj2/command/Command.java

@@ -1,3 +1,6 @@
+Skipping builds for arm32 (toolchain is marked optional)
+Skipping builds for arm64 (toolchain is marked optional)
+Skipping builds for roboRio (toolchain is marked optional)
 // Copyright (c) FIRST and other WPILib contributors.
 // Open Source Software; you can modify and/or share it under the terms of
 // the WPILib BSD license file in the root directory of this project.
@@ -15,7 +18,6 @@
 import java.util.Collection;
 import java.util.HashSet;
 import java.util.Set;
-import java.util.concurrent.atomic.AtomicInteger;
 import java.util.function.BooleanSupplier;
 
 /**
@@ -406,24 +408,21 @@ public RepeatCommand repeatedly() {
    * Decorates this command to run repeatedly, restarting until the command runs for the given times
    * amount. The decorated command can still be canceled.
    *
-   * <p>This is just syntactic sugar for {@code this.finallyDo(() ->
-   * counter[0]++).repeatedly().until(() -> counter[0] >= times)} which just means: runs the
-   * command, then increments the counter, repeatedly until the counter saturates.
-   *
    * <p>Note: This decorator works by adding this command to a composition. The command the
    * decorator was called on cannot be scheduled independently or be added to a different
    * composition (namely, decorators), unless it is manually cleared from the list of composed
    * commands with {@link CommandScheduler#removeComposedCommand(Command)}. The command composition
    * returned from this method can be further decorated without issue.
    *
-   * @param times the count of times to run the command (inclusively).
+   * @param repetitions the count of times to run the command.
    * @return the decorated command
    */
-  public ParallelRaceGroup repeatedly(int times) {
-    AtomicInteger counter = new AtomicInteger(0);
-    return this.finallyDo(counter::getAndIncrement)
-        .repeatedly()
-        .until(() -> counter.get() >= times);
+  public ParallelRaceGroup repeatedly(int repetitions) {
+    // use an array so that it stays in the heap instead of stack.
+    // We use an array with a size of 1 instead of `AtomicInteger` because of the performance
+    // overhead
+    int[] counter = new int[1];
+    return this.finallyDo(() -> counter[0]++).repeatedly().until(() -> counter[0] >= repetitions);
   }
 
   /**