Daniel/gloves/reintegrate foreach fix

VortexEngine/src/Leds/LedTypes.h

@@ -160,7 +160,7 @@ typedef uint64_t LedMap;
 #define MAP_IS_ONE_LED(map) (map && !(map & (map-1)))
 
 // foreach led macro (only iterates singles)
-#define MAP_FOREACH_LED(map) for (LedPos pos = ledmapGetFirstLed(map); pos < LED_COUNT; pos = ledmapGetNextLed(map, pos))
+#define MAP_FOREACH_LED(map) for (LedPos pos = ledmapGetFirstLed(map); pos != LED_COUNT; pos = ledmapGetNextLed(map, pos))
 
 // convert a map to the first Led position in the map
 inline LedPos ledmapGetFirstLed(LedMap map)

VortexEngine/src/Menus/Menu.cpp

@@ -146,6 +146,13 @@ void Menu::showExit()
   Leds::setIndex(THUMB_TOP, RGB_WHITE0);
 }
 
+void Menu::bypassLedSelection(LedMap map)
+{
+  m_ledSelected = true;
+  m_targetLeds = map;
+  onLedSelected();
+}
+
 void Menu::onLedSelected()
 {
 }

VortexEngine/src/Menus/Menu.h

@@ -45,6 +45,8 @@ class Menu
   // blink the selected finger
   virtual void blinkSelection(uint32_t offMs = 250, uint32_t onMs = 500);
 
+  // skip led selection
+  void bypassLedSelection(LedMap map);
   // an overridable api that allows derived menus to decide which led selections
   // should be available before they have actually opened
   virtual bool isValidLedSelection(LedMap selection) const { return true; }

VortexEngine/src/Menus/MenuList/ColorSelect.cpp

@@ -48,8 +48,7 @@ bool ColorSelect::init()
     return false;
   }
   if (cur->isMultiLed()) {
-    m_targetLeds = MAP_LED(LED_MULTI);
-    m_ledSelected = true;
+    bypassLedSelection(MAP_LED(LED_MULTI));
   }
   m_state = STATE_INIT;
   DEBUG_LOG("Entered color select");

VortexEngine/src/Menus/MenuList/Randomizer.cpp

@@ -161,11 +161,14 @@ bool Randomizer::reRoll()
     if (!reRollMulti()) {
       return false;
     }
+  } else if (!reRollSingles()) {
+    return false;
   }
-#endif
+#else
   if (!reRollSingles()) {
     return false;
   }
+#endif
   // initialize the mode with the new pattern and colorset
   m_previewMode.init();
   return true;