From 929847209d61d797ca38edfdd0cd411e82a40205 Mon Sep 17 00:00:00 2001 From: Risto Date: Thu, 2 Dec 2021 01:28:13 +0100 Subject: [PATCH] Fixed compiler warnings (#41) Fixed 4 compiler warnings about comparison of signed and unsigned int. --- radio/src/thirdparty/libopenui/src/menu.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/radio/src/thirdparty/libopenui/src/menu.cpp b/radio/src/thirdparty/libopenui/src/menu.cpp index e4c6ce189..28d46105e 100644 --- a/radio/src/thirdparty/libopenui/src/menu.cpp +++ b/radio/src/thirdparty/libopenui/src/menu.cpp @@ -27,7 +27,7 @@ int MenuBody::rangeCheck(int index) { if (index < 0) index = lines.size() - 1; - else if (index > lines.size() - 1) + else if (index > (signed)lines.size() - 1) index = 0; return index; @@ -132,7 +132,7 @@ bool MenuBody::onTouchEnd(coord_t /*x*/, coord_t y) // dynamically find the selected element based on variable line heights coord_t yAccumulator = 0; - for (int i = 0; i < lines.size(); i++) { + for (int i = 0; i < (signed)lines.size(); i++) { if (y >= yAccumulator && y <= yAccumulator + lines[i].lineHeight()) { index = i; break; @@ -169,7 +169,7 @@ void MenuBody::paint(BitmapBuffer * dc) dc->clear(COLOR_THEME_PRIMARY2); int y = 0; - for (int i = 0; i < lines.size(); i++) { + for (int i = 0; i < (signed)lines.size(); i++) { auto& line = lines[i]; LcdFlags flags = COLOR_THEME_PRIMARY3 | MENU_FONT; @@ -238,7 +238,7 @@ void Menu::updatePosition() { // calcualte the correct menu height given that the line heights are variable coord_t height = 0; - for (int i = 0; i < content->body.lines.size(); i++) + for (int i = 0; i < (signed)content->body.lines.size(); i++) height += content->body.lines[i].lineHeight(); if (!toolbar) {