From 445c8eade158e521ad492cf4039ba024f419a8a1 Mon Sep 17 00:00:00 2001 From: Adam Honse Date: Wed, 7 Aug 2024 21:38:55 -0500 Subject: [PATCH] Fix signed/unsigned mismatch warning in RGBController_LogitechG915.cpp --- .../RGBController_LogitechG915.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/Controllers/LogitechController/LogitechG915Controller/RGBController_LogitechG915.cpp b/Controllers/LogitechController/LogitechG915Controller/RGBController_LogitechG915.cpp index 77508c84..03f91e45 100644 --- a/Controllers/LogitechController/LogitechG915Controller/RGBController_LogitechG915.cpp +++ b/Controllers/LogitechController/LogitechG915Controller/RGBController_LogitechG915.cpp @@ -393,7 +393,7 @@ void RGBController_LogitechG915::DeviceUpdateLEDs() zone = ( leds[led_idx].value >> 8 ); idx = ( leds[led_idx].value ); - if (current_colors[led_idx]==new_colors[led_idx]) + if(current_colors[led_idx]==new_colors[led_idx]) { /*-------------------------------------------------*\ | Don't send if key color is not changed | @@ -401,7 +401,7 @@ void RGBController_LogitechG915::DeviceUpdateLEDs() continue; } - switch (zone) + switch(zone) { case LOGITECH_G915_ZONE_MODE_GKEYS: idx = ((idx & 0x00ff) + 0xb3); @@ -426,7 +426,7 @@ void RGBController_LogitechG915::DeviceUpdateLEDs() colorkey = new_colors[led_idx]; - if (ledsByColors.count(colorkey) == 0) + if(ledsByColors.count(colorkey) == 0) { ledsByColors.insert(std::pair>(colorkey, {})); } @@ -462,14 +462,14 @@ void RGBController_LogitechG915::DeviceUpdateLEDs() for(uint8_t i = 0; i < max_key_per_color; i++) { - if(bi + i < x.second.size()) + if((bi + i) < (uint8_t)x.second.size()) { frame_buffer_big_mode[frame_pos] = x.second[bi+i]; frame_pos++; } } - if (frame_pos < data_size) + if(frame_pos < data_size) { /*-----------------------------------------*\ | Zeroing just what is needed and if needed | @@ -506,7 +506,7 @@ void RGBController_LogitechG915::DeviceUpdateLEDs() li++; led_in_little_frame++; - if (led_in_little_frame == 4) + if(led_in_little_frame == 4) { /*-----------------------------------------*\ | No End of Data byte if the packet is full |