Fix -Wtype-limits warning

actual has type Uint8, so there's no need to compare against 0.
smcv · Sep 9, 2024 · c797ae1 · c797ae1
1 parent 6e885d9
commit c797ae1
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/test/testautomation_surface.c b/test/testautomation_surface.c
@@ -1339,15 +1339,15 @@ static int SDLCALL surface_testPalettization(void *arg)
         int idx = i;
         Uint8 actual = pixels[idx];
         Uint8 expected = colors[i].e;
-        SDLTest_AssertCheck(0 <= actual && actual < SDL_arraysize(palette_colors), "0 <= output->pixels[%d] < %d", idx, (int)SDL_arraysize(palette_colors));
+        SDLTest_AssertCheck(actual < SDL_arraysize(palette_colors), "output->pixels[%d] < %d", idx, (int)SDL_arraysize(palette_colors));
         SDLTest_AssertCheck(actual == expected, "Expected output->pixels[%d] == %u, got %u", idx, expected, actual);
     }
     SDLTest_AssertPass("Check palette 1:1 mapping");
     for (i = 0; i < SDL_arraysize(palette_colors); i++) {
         int idx = SDL_arraysize(colors) + i;
         Uint8 actual = pixels[idx];
         Uint8 expected = i;
-        SDLTest_AssertCheck(0 <= actual && actual < SDL_arraysize(palette_colors), "0 <= output->pixels[%d] < %d", idx, (int)SDL_arraysize(palette_colors));
+        SDLTest_AssertCheck(actual < SDL_arraysize(palette_colors), "output->pixels[%d] < %d", idx, (int)SDL_arraysize(palette_colors));
         SDLTest_AssertCheck(actual == expected, "Expected output->pixels[%d] == %u, got %u", idx, expected, actual);
     }
     SDL_DestroyPalette(palette);