From f1e2f68a2028d4a901bd0a2401fa5e4bf6e3e847 Mon Sep 17 00:00:00 2001 From: Evy Bongers Date: Sun, 26 May 2024 20:57:00 +0200 Subject: [PATCH] Add todo for validating the colormap index Signed-off-by: Evy Bongers --- .../src/kaleidoscope/plugin/Colormap-Overlay.h | 4 +++- 1 file changed, 3 insertions(+), 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 index d560a610b..757868dce 100644 --- 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_++; } }