Skip to content

Commit 97cea28

Browse files
committed
Combined conditions
1 parent 41270f3 commit 97cea28

File tree

1 file changed

+10
-11
lines changed

1 file changed

+10
-11
lines changed

src/_imagingcms.c

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -213,32 +213,31 @@ cms_transform_dealloc(CmsTransformObject *self) {
213213

214214
static cmsUInt32Number
215215
findLCMStype(char *PILmode) {
216-
if (strcmp(PILmode, "RGB") == 0) {
217-
return TYPE_RGBA_8;
218-
} else if (strcmp(PILmode, "RGBA") == 0) {
219-
return TYPE_RGBA_8;
220-
} else if (strcmp(PILmode, "RGBX") == 0) {
216+
if (
217+
strcmp(PILmode, "RGB") == 0 ||
218+
strcmp(PILmode, "RGBA") == 0 ||
219+
strcmp(PILmode, "RGBX") == 0
220+
) {
221221
return TYPE_RGBA_8;
222222
} else if (strcmp(PILmode, "RGBA;16B") == 0) {
223223
return TYPE_RGBA_16;
224224
} else if (strcmp(PILmode, "CMYK") == 0) {
225225
return TYPE_CMYK_8;
226-
} else if (strcmp(PILmode, "L") == 0) {
227-
return TYPE_GRAY_8;
228226
} else if (strcmp(PILmode, "L;16") == 0) {
229227
return TYPE_GRAY_16;
230228
} else if (strcmp(PILmode, "L;16B") == 0) {
231229
return TYPE_GRAY_16_SE;
232-
} else if (strcmp(PILmode, "YCCA") == 0) {
233-
return TYPE_YCbCr_8;
234-
} else if (strcmp(PILmode, "YCC") == 0) {
230+
} else if (
231+
strcmp(PILmode, "YCCA") == 0 ||
232+
strcmp(PILmode, "YCC") == 0
233+
) {
235234
return TYPE_YCbCr_8;
236235
} else if (strcmp(PILmode, "LAB") == 0) {
237236
// LabX equivalent like ALab, but not reversed -- no #define in lcms2
238237
return (COLORSPACE_SH(PT_LabV2) | CHANNELS_SH(3) | BYTES_SH(1) | EXTRA_SH(1));
239238
}
240239
else {
241-
/* take a wild guess... */
240+
/* presume "L" by default */
242241
return TYPE_GRAY_8;
243242
}
244243
}

0 commit comments

Comments
 (0)