Merge pull request #26 from technototes/BugFix

Fixed a hard size limitation on logging entries. Hopefully no more crashing

RobotLibrary/src/main/java/com/technototes/library/logger/Logger.java

@@ -11,6 +11,7 @@
 import java.lang.reflect.Field;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
+import java.sql.Array;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.LinkedHashSet;
@@ -89,13 +90,17 @@ private void configure(Object root) {
     }
 
     // TODO make list and do sort with comparators
-    // List<List<Entry<?>>
+    // I wish this had a comment describing what Alex thinks it's doing,
+    // I *think* it'strying to set the 'indexed' entries to their preferred locations
+    // then filling in the gaps with unindexed  or lower priority entries.
+    // That bottom loop is also quite slow, but we're talking about 0-20 entries, so performance
+    // is probably irrelevant...
     private Entry<?>[] generate(Set<Entry<?>> a) {
-        Entry<?>[] returnEntry = new Entry[10];
+        Entry<?>[] returnEntry = new Entry[a.size()];
         List<Entry<?>> unindexed = new ArrayList<>();
         for (Entry<?> e : a) {
             int index = e.getIndex();
-            if (index != -1) {
+            if (index >= 0 && index < returnEntry.length) {
                 Entry<?> other = returnEntry[index];
                 if (other == null) {
                     returnEntry[index] = e;
@@ -116,7 +121,6 @@ private Entry<?>[] generate(Set<Entry<?>> a) {
                 returnEntry[i] = unindexed.remove(0);
             }
         }
-
         return returnEntry;
     }