Add todo for validating the colormap index

Signed-off-by: Evy Bongers <[email protected]>
keyboardio · May 26, 2024 · f1e2f68 · f1e2f68
1 parent bea3089
commit f1e2f68
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/plugins/Kaleidoscope-Colormap-Overlay/src/kaleidoscope/plugin/Colormap-Overlay.h b/plugins/Kaleidoscope-Colormap-Overlay/src/kaleidoscope/plugin/Colormap-Overlay.h
@@ -59,7 +59,9 @@ class ColormapOverlay : public kaleidoscope::Plugin {
     overlay_count_ = 0;
     for (int layer_ = 0; layer_ < _layer_count; layer_++) {
       for (int key_index_ = 0; key_index_ < kaleidoscope_internal::device.matrix_rows * kaleidoscope_internal::device.matrix_columns; key_index_++) {
-        overlays_[overlay_count_] = Overlay(layer_, /* TODO(EvyBongers) */, overlays[layer_][key_index_]);
+        color_index_ = overlays[layer_][key_index_];
+        // TODO(EvyBongers): validate the color index?
+        overlays_[overlay_count_] = Overlay(layer_, /* TODO(EvyBongers) */, color_index_);
         overlay_count_++;
       }
     }