search

karagenit / allwpilib

Official Repository of WPILibJ and WPILibC
Other
0 stars 0 forks source link

Java Collections #5

Open karagenit opened 6 years ago

karagenit commented 6 years ago

See wpilibsuite/allwpilib#507

Vector

./wpilibj/src/athena/java/edu/wpi/first/wpilibj/Preferences.java:10:import java.util.Vector;
./wpilibj/src/shared/java/edu/wpi/first/wpilibj/command/CommandGroup.java:11:import java.util.Vector;
./wpilibj/src/shared/java/edu/wpi/first/wpilibj/command/Scheduler.java:12:import java.util.Vector;
./wpilibj/src/shared/java/edu/wpi/first/wpilibj/command/Set.java:11:import java.util.Vector;
./wpilibj/src/shared/java/edu/wpi/first/wpilibj/livewindow/LiveWindow.java:12:import java.util.Vector;
./wpilibj/src/shared/java/edu/wpi/first/wpilibj/util/SortedVector.java:10:import java.util.Vector;
./wpilibj/src/sim/java/edu/wpi/first/wpilibj/Preferences.java:17:import java.util.Vector;

Hashtable

./wpilibj/src/athena/java/edu/wpi/first/wpilibj/CameraServer.java:29:import java.util.Hashtable;
./wpilibj/src/shared/java/edu/wpi/first/wpilibj/command/Scheduler.java:11:import java.util.Hashtable;
./wpilibj/src/shared/java/edu/wpi/first/wpilibj/livewindow/LiveWindow.java:11:import java.util.Hashtable;
./wpilibj/src/shared/java/edu/wpi/first/wpilibj/smartdashboard/SmartDashboard.java:11:import java.util.Hashtable;
./wpilibj/src/sim/java/edu/wpi/first/wpilibj/Preferences.java:16:import java.util.Hashtable;

LinkedListElement.java and Set.java could be replaced by ArrayList and Java Set/Hashset.

karagenit commented 6 years ago

Enumerations are also used with Vectors, which should be replaced with Iterators.

karagenit commented 6 years ago

Oops, apparently this is already being worked on, see PR #508