From 00ffab8f6259191efa08e1c5620af2b497489921 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gu=C3=B0ni=20M=C3=A1r=20Gilbert?= Date: Sun, 9 Apr 2023 13:51:28 +0000 Subject: [PATCH] optimisation: combine if statements in Sensors menu FINDA status is always 0 or 1 Change in memory: Flash: -28 bytes SRAM: 0 bytes --- Firmware/ultralcd.cpp | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/Firmware/ultralcd.cpp b/Firmware/ultralcd.cpp index 82c72a5b8..825e7e778 100644 --- a/Firmware/ultralcd.cpp +++ b/Firmware/ultralcd.cpp @@ -3297,20 +3297,15 @@ static void lcd_show_sensors_state() { //0: N/A; 1: OFF; 2: ON uint8_t pinda_state = STATE_NA; - uint8_t finda_state = STATE_NA; uint8_t idler_state = STATE_NA; pinda_state = READ(Z_MIN_PIN); - if (MMU2::mmu2.Enabled()) - { - finda_state = MMU2::mmu2.FindaDetectsFilament(); - } lcd_puts_at_P(0, 0, MSG_PINDA); lcd_set_cursor(LCD_WIDTH - 14, 0); lcd_print_state(pinda_state); - if (MMU2::mmu2.Enabled()) - { + if (MMU2::mmu2.Enabled()) { + const uint8_t finda_state = MMU2::mmu2.FindaDetectsFilament(); lcd_puts_at_P(10, 0, _n("FINDA"));////MSG_FINDA c=5 lcd_set_cursor(LCD_WIDTH - 3, 0); lcd_print_state(finda_state);