commit c02cae89e542f4ee18f7de9f66ff867a493644cf
author Wan-Teh Chang <wtc@google.com> Wed Jul 10 14:55:44 2024 -0700
committer Wan-Teh Chang <wtc@google.com> Wed Jul 10 15:52:10 2024 -0700
tree e42a2f356962f9b2350d20bbcd5a2d9ce028dcf4
parent 084eac1aaf30000c30bd733caf1046a652102620

Remove unneeded uint32_t casts

avif->width and avif->height are of the uint32_t type, so it is not
necessary to cast them to uint32_t.

apps/shared/avifjpeg.c

diff --git a/apps/shared/avifjpeg.c b/apps/shared/avifjpeg.c
index 7486cb4..6b00681 100644
--- a/apps/shared/avifjpeg.c
+++ b/apps/shared/avifjpeg.c
@@ -55,7 +55,7 @@
 
     avif->width = cinfo->image_width;
     avif->height = cinfo->image_height;
-    if ((uint32_t)avif->width > sizeLimit / (uint32_t)avif->height) {
+    if (avif->width > sizeLimit / avif->height) {
         return AVIF_FALSE;
     }
 

apps/shared/avifpng.c

diff --git a/apps/shared/avifpng.c b/apps/shared/avifpng.c
index d0e7450..a38cfab 100644
--- a/apps/shared/avifpng.c
+++ b/apps/shared/avifpng.c
@@ -456,7 +456,7 @@
         fprintf(stderr, "png_get_channels() should return 3 or 4 but returns %d.\n", numChannels);
         goto cleanup;
     }
-    if ((uint32_t)avif->width > imageSizeLimit / (uint32_t)avif->height) {
+    if (avif->width > imageSizeLimit / avif->height) {
         fprintf(stderr, "Too big PNG dimensions (%d x %d > %u px): %s\n", avif->width, avif->height, imageSizeLimit, inputFilename);
         goto cleanup;
     }