Skip to content

Commit adfb66f

Browse files
committed
Fix Compliation errors from rebase
1 parent 84aa437 commit adfb66f

File tree

6 files changed

+10
-14
lines changed

6 files changed

+10
-14
lines changed

src/_imaging.c

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -297,6 +297,7 @@ ExportArrowArrayPyCapsule(ImagingObject *self) {
297297
static PyObject *
298298
_new_arrow(PyObject *self, PyObject *args) {
299299
char *mode;
300+
ModeID mode_id;
300301
int xsize, ysize;
301302
PyObject *schema_capsule, *array_capsule;
302303
PyObject *ret;
@@ -307,9 +308,11 @@ _new_arrow(PyObject *self, PyObject *args) {
307308
return NULL;
308309
}
309310

311+
mode_id = findModeID(mode);
312+
310313
// ImagingBorrowArrow is responsible for retaining the array_capsule
311314
ret = PyImagingNew(
312-
ImagingNewArrow(mode, xsize, ysize, schema_capsule, array_capsule)
315+
ImagingNewArrow(mode_id, xsize, ysize, schema_capsule, array_capsule)
313316
);
314317
if (!ret) {
315318
return ImagingError_ValueError("Invalid Arrow array mode or size mismatch");

src/libImaging/BcnEncode.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,7 @@ int
253253
ImagingBcnEncode(Imaging im, ImagingCodecState state, UINT8 *buf, int bytes) {
254254
int n = state->state;
255255
int has_alpha_channel =
256-
strcmp(im->mode, "RGBA") == 0 || strcmp(im->mode, "LA") == 0;
256+
im->mode == IMAGING_MODE_RGBA || im->mode == IMAGING_MODE_LA;
257257

258258
UINT8 *dst = buf;
259259

src/libImaging/Convert.c

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1517,9 +1517,6 @@ static struct {
15171517
{IMAGING_MODE_RGB, IMAGING_MODE_I_16N, rgb2i16l},
15181518
#endif
15191519
{IMAGING_MODE_RGB, IMAGING_MODE_F, rgb2f},
1520-
{IMAGING_MODE_RGB, IMAGING_MODE_BGR_15, rgb2bgr15},
1521-
{IMAGING_MODE_RGB, IMAGING_MODE_BGR_16, rgb2bgr16},
1522-
{IMAGING_MODE_RGB, IMAGING_MODE_BGR_24, rgb2bgr24},
15231520
{IMAGING_MODE_RGB, IMAGING_MODE_RGBA, rgb2rgba},
15241521
{IMAGING_MODE_RGB, IMAGING_MODE_RGBa, rgb2rgba},
15251522
{IMAGING_MODE_RGB, IMAGING_MODE_RGBX, rgb2rgba},

src/libImaging/Jpeg2KEncode.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -332,7 +332,7 @@ j2k_encode_entry(Imaging im, ImagingCodecState state) {
332332
pack = j2k_pack_rgba;
333333
#if ((OPJ_VERSION_MAJOR == 2 && OPJ_VERSION_MINOR == 5 && OPJ_VERSION_BUILD >= 3) || \
334334
(OPJ_VERSION_MAJOR == 2 && OPJ_VERSION_MINOR > 5) || OPJ_VERSION_MAJOR > 2)
335-
} else if (strcmp(im->mode, "CMYK") == 0) {
335+
} else if (im->mode == IMAGING_MODE_CMYK) {
336336
components = 4;
337337
color_space = OPJ_CLRSPC_CMYK;
338338
pack = j2k_pack_rgba;

src/libImaging/Pack.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -651,9 +651,7 @@ static struct {
651651
{IMAGING_MODE_I_16, IMAGING_RAWMODE_I_16N, 16, packI16N_I16
652652
}, // LibTiff native->image endian.
653653
{IMAGING_MODE_I_16L, IMAGING_RAWMODE_I_16N, 16, packI16N_I16},
654-
{IMAGING_MODE_I_16B, IMAGING_RAWMODE_I_16N, 16, packI16N_I16B},
655-
656-
{NULL} /* sentinel */
654+
{IMAGING_MODE_I_16B, IMAGING_RAWMODE_I_16N, 16, packI16N_I16B}
657655
};
658656

659657
ImagingShuffler

src/libImaging/Unpack.c

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1313,7 +1313,7 @@ copy4skip2(UINT8 *_out, const UINT8 *in, int pixels) {
13131313
/* Unpack to "I" and "F" images */
13141314

13151315
#define UNPACK_RAW(NAME, GET, INTYPE, OUTTYPE) \
1316-
static void NAME(UINT8 *out, const UINT8 *in, int pixels) { \
1316+
static void NAME(UINT8 *out_, const UINT8 *in, int pixels) { \
13171317
int i; \
13181318
OUTTYPE *out = (OUTTYPE *)out_; \
13191319
for (i = 0; i < pixels; i++, in += sizeof(INTYPE)) { \
@@ -1322,7 +1322,7 @@ copy4skip2(UINT8 *_out, const UINT8 *in, int pixels) {
13221322
}
13231323

13241324
#define UNPACK(NAME, COPY, INTYPE, OUTTYPE) \
1325-
static void NAME(UINT8 *out, const UINT8 *in, int pixels) { \
1325+
static void NAME(UINT8 *out_, const UINT8 *in, int pixels) { \
13261326
int i; \
13271327
OUTTYPE *out = (OUTTYPE *)out_; \
13281328
INTYPE tmp_; \
@@ -1839,9 +1839,7 @@ static struct {
18391839

18401840
{IMAGING_MODE_I_16, IMAGING_RAWMODE_I_16R, 16, unpackI16R_I16},
18411841

1842-
{IMAGING_MODE_I_16, IMAGING_RAWMODE_I_12, 12, unpackI12_I16}, // 12 bit Tiffs stored in 16bits.
1843-
1844-
{NULL} /* sentinel */
1842+
{IMAGING_MODE_I_16, IMAGING_RAWMODE_I_12, 12, unpackI12_I16} // 12 bit Tiffs stored in 16bits.
18451843
};
18461844

18471845
ImagingShuffler

0 commit comments

Comments
 (0)