try

radarhere · Oct 7, 2023 · 9e75715 · 9e75715
1 parent 35b95df
commit 9e75715
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/Tests/test_lib_image.py b/Tests/test_lib_image.py
@@ -5,6 +5,8 @@
 from PIL import Image
 
 mode_names = (
+    "BGR;15",
+    "BGR;16",
     "BGR;24",
 )
 

diff --git a/src/_imaging.c b/src/_imaging.c
@@ -3760,9 +3760,10 @@ _compare_pixels(
     // Fortunately, all of the modes that have extra bytes in their pixels use four bytes for their pixels.
     UINT32 mask = 0xffffffff;
     if (
-        !strcmp(mode, "RGB") || !strcmp(mode, "YCbCr") ||
-        !strcmp(mode, "HSV") || !strcmp(mode, "LAB")
+        !strcmp(mode, "RGB") || !strcmp(mode, "BGR;24") ||
+        !strcmp(mode, "YCbCr") || !strcmp(mode, "HSV") || !strcmp(mode, "LAB")
     ) {
+        printf("threeinfour\n");
         // These modes have three channels in four bytes,
         // so we have to ignore the last byte.
 #ifdef WORDS_BIGENDIAN
@@ -3774,13 +3775,17 @@ _compare_pixels(
         // These modes have two channels in four bytes,
         // so we have to ignore the middle two bytes.
         mask = 0xff0000ff;
+        printf("twoinfour\n");
+    } else {
+        printf("else\n");
     }
 
     if (mask == 0xffffffff) {
         // If we aren't masking anything we can use memcmp.
         int y;
         for (y = 0; y < ysize; y++) {
             if (memcmp(pixels_a[y], pixels_b[y], linesize)) {
+                printf("full fail\n");
                 return 1;
             }
         }
@@ -3792,11 +3797,13 @@ _compare_pixels(
             UINT32 *line_b = (UINT32*)pixels_b[y];
             for (x = 0; x < xsize; x++, line_a++, line_b++) {
                 if ((*line_a & mask) != (*line_b & mask)) {
+                    printf("mask fail linea %d lineb %d mask %d lineamask %d linebmask %d\n", *line_a, *line_b, mask, *line_a & mask, *line_b & mask);
                     return 1;
                 }
             }
         }
     }
+    printf("success %d\n", mask);
     return 0;
 }