Randomize could create crash

kockie69 · Dec 1, 2021 · b0a7384 · b0a7384
1 parent d0b6c8a
commit b0a7384
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/src/ctrl/WaveformSwitch.h b/src/ctrl/WaveformSwitch.h
@@ -83,18 +83,19 @@ inline void WaveformSwitch::onButton(const event::Button& e) {
 }
 
 void WaveformSwitch::drawLayer(const DrawArgs& args,int layer) {
-	    if (layer ==1) 
-	        ::rack::ParamWidget::draw(args);
-        ParamWidget::drawLayer(args,layer);
-        }
+	if (layer ==1) 
+	    ::rack::ParamWidget::draw(args);
+    ParamWidget::drawLayer(args,layer);
+}
 
 WaveCell* WaveformSwitch::getCell(int index) {
     return cells[index];
 }
 
 void WaveformSwitch::step() {
     float fval = SqHelper::getValue(this);
-    int val = int(std::round(fval));
+    //int val = int(std::round(fval));
+    int val = int(std::floor(fval));
     if (val != currentValue) {
         //DEBUG("step found new value %d old=%d\n", val, currentValue);
         if (currentValue >= 0) {
@@ -104,9 +105,11 @@ void WaveformSwitch::step() {
             }
         }
         auto cell = getCell(val);
-        cell->setState(true);
-        currentValue = val;
-        fw->dirty = true;
+        if (cell) {
+            cell->setState(true);
+            currentValue = val;
+            fw->dirty = true;
+        }
     }
     ParamWidget::step();
 }