piotr/edgetx-gx12
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed compiler warnings (#41)

Browse Source 
Fixed 4 compiler warnings about comparison of signed and unsigned int.
This commit is contained in:
Risto
2021-12-02 11:28:13 +11:00
committed by GitHub
parent faae2fff8c
commit 929847209d
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
radio/src/thirdparty/libopenui/src/menu.cpp
Vendored
+4 -4
View File
@@ -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) {