From c5cf9692adb996351fc13b5d5d1ae957b597384a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Esmit=20Pe=CC=81rez=20C?= Date: Fri, 12 Sep 2014 23:16:08 -0500 Subject: [PATCH] Documentation changes Removed comments regarding design decisions. MicroViewSlider:draw() retested to ensure refactoring did not introduce regressions. --- MicroView.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/MicroView.cpp b/MicroView.cpp index de90216..22835f1 100644 --- a/MicroView.cpp +++ b/MicroView.cpp @@ -1467,11 +1467,10 @@ MicroViewSlider::MicroViewSlider(uint8_t newx, uint8_t newy, int16_t min, int16_ /** \brief MicroViewSlider class initialisation with style. - Initialise the MicroViewSlider widget with style WIDGETSTYLE0 or WIDGETSTYLE1 or WIDGETSTYLE2 (like 0, but vertical) or WIDGETSTYLE3 (like 1, but vertical). If this list gets any longer, it might be better as a switch/case statement. + Initialise the MicroViewSlider widget with style WIDGETSTYLE0 or WIDGETSTYLE1 or WIDGETSTYLE2 (like 0, but vertical) or WIDGETSTYLE3 (like 1, but vertical). */ MicroViewSlider::MicroViewSlider(uint8_t newx, uint8_t newy, int16_t min, int16_t max, uint8_t sty):MicroViewWidget(newx, newy, min, max) { - // @esmitperez: replaced with switch...but original if/else here because it's faster? switch(sty){ case WIDGETSTYLE1: style=1; @@ -1584,7 +1583,6 @@ void MicroViewSlider::draw() { } // Draw value - // @esmitperez: replaced with switch...but original if/else here because it's faster? switch(style){ case 0: uView.setCursor(offsetX+totalTicks+4, offsetY+1);