remove todo in java getStrings

arrays are mutable refs so a new array must be constructed
rzblue · Oct 24, 2024 · 68f4d95 · 68f4d95
1 parent 7ccf7ec
commit 68f4d95
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/wpilibj/src/main/java/edu/wpi/first/wpilibj/Alert.java b/wpilibj/src/main/java/edu/wpi/first/wpilibj/Alert.java
@@ -164,9 +164,6 @@ public Set<PublishedAlert> getSetForType(AlertType type) {
     }
 
     private String[] getStrings(AlertType type) {
-      // TODO: some optimizations available here- could iterate explicitly and cache the array for
-      // potential reuse (if size is same, refill array, if nothing has changed, return original
-      // array)
       return getSetForType(type).stream().map(a -> a.text()).toArray(String[]::new);
     }