summaryrefslogtreecommitdiff
path: root/source/fitz
diff options
context:
space:
mode:
Diffstat (limited to 'source/fitz')
-rw-r--r--source/fitz/colorspace.c90
1 files changed, 40 insertions, 50 deletions
diff --git a/source/fitz/colorspace.c b/source/fitz/colorspace.c
index 89bef0aa..80d15157 100644
--- a/source/fitz/colorspace.c
+++ b/source/fitz/colorspace.c
@@ -953,11 +953,10 @@ static void fast_gray_to_rgb(fz_context *ctx, fz_pixmap *dst, fz_pixmap *src, fz
ptrdiff_t s_line_inc = src->stride - w * sn;
/* If copying spots, they must match, and we can never drop alpha (but we can invent it) */
- if ((copy_spots && ss != ds) || (!da && sa))
- {
- assert("This should never happen" == NULL);
- fz_throw(ctx, FZ_ERROR_GENERIC, "Cannot convert between incompatible pixmaps");
- }
+ if (copy_spots && ss != ds)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "incompatible number of spots when converting pixmap");
+ if (!da && sa)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "cannot drop alpha when converting pixmap");
if ((int)w < 0 || h < 0)
return;
@@ -1088,11 +1087,10 @@ static void fast_gray_to_cmyk(fz_context *ctx, fz_pixmap *dst, fz_pixmap *src, f
ptrdiff_t s_line_inc = src->stride - w * sn;
/* If copying spots, they must match, and we can never drop alpha (but we can invent it) */
- if ((copy_spots && ss != ds) || (!da && sa))
- {
- assert("This should never happen" == NULL);
- fz_throw(ctx, FZ_ERROR_GENERIC, "Cannot convert between incompatible pixmaps");
- }
+ if (copy_spots && ss != ds)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "incompatible number of spots when converting pixmap");
+ if (!da && sa)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "cannot drop alpha when converting pixmap");
if ((int)w < 0 || h < 0)
return;
@@ -1228,11 +1226,10 @@ static void fast_rgb_to_gray(fz_context *ctx, fz_pixmap *dst, fz_pixmap *src, fz
ptrdiff_t s_line_inc = src->stride - w * sn;
/* If copying spots, they must match, and we can never drop alpha (but we can invent it) */
- if ((copy_spots && ss != ds) || (!da && sa))
- {
- assert("This should never happen" == NULL);
- fz_throw(ctx, FZ_ERROR_GENERIC, "Cannot convert between incompatible pixmaps");
- }
+ if (copy_spots && ss != ds)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "incompatible number of spots when converting pixmap");
+ if (!da && sa)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "cannot drop alpha when converting pixmap");
if ((int)w < 0 || h < 0)
return;
@@ -1353,11 +1350,10 @@ static void fast_bgr_to_gray(fz_context *ctx, fz_pixmap *dst, fz_pixmap *src, fz
ptrdiff_t s_line_inc = src->stride - w * sn;
/* If copying spots, they must match, and we can never drop alpha (but we can invent it) */
- if ((copy_spots && ss != ds) || (!da && sa))
- {
- assert("This should never happen" == NULL);
- fz_throw(ctx, FZ_ERROR_GENERIC, "Cannot convert between incompatible pixmaps");
- }
+ if (copy_spots && ss != ds)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "incompatible number of spots when converting pixmap");
+ if (!da && sa)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "cannot drop alpha when converting pixmap");
if ((int)w < 0 || h < 0)
return;
@@ -1481,11 +1477,10 @@ static void fast_rgb_to_cmyk(fz_context *ctx, fz_pixmap *dst, fz_pixmap *src, fz
ptrdiff_t s_line_inc = src->stride - w * sn;
/* Spots must match, and we can never drop alpha (but we can invent it) */
- if ((copy_spots || ss != ds) || (!da && sa))
- {
- assert("This should never happen" == NULL);
- fz_throw(ctx, FZ_ERROR_GENERIC, "Cannot convert between incompatible pixmaps");
- }
+ if (copy_spots && ss != ds)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "incompatible number of spots when converting pixmap");
+ if (!da && sa)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "cannot drop alpha when converting pixmap");
if ((int)w < 0 || h < 0)
return;
@@ -1641,11 +1636,10 @@ static void fast_bgr_to_cmyk(fz_context *ctx, fz_pixmap *dst, fz_pixmap *src, fz
ptrdiff_t s_line_inc = src->stride - w * sn;
/* Spots must match, and we can never drop alpha (but we can invent it) */
- if ((copy_spots && ss != ds) || (!da && sa))
- {
- assert("This should never happen" == NULL);
- fz_throw(ctx, FZ_ERROR_GENERIC, "Cannot convert between incompatible pixmaps");
- }
+ if (copy_spots && ss != ds)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "incompatible number of spots when converting pixmap");
+ if (!da && sa)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "cannot drop alpha when converting pixmap");
if ((int)w < 0 || h < 0)
return;
@@ -1801,11 +1795,10 @@ static void fast_cmyk_to_gray(fz_context *ctx, fz_pixmap *dst, fz_pixmap *src, f
ptrdiff_t s_line_inc = src->stride - w * sn;
/* Spots must match, and we can never drop alpha (but we can invent it) */
- if ((copy_spots && ss != ds) || (!da && sa))
- {
- assert("This should never happen" == NULL);
- fz_throw(ctx, FZ_ERROR_GENERIC, "Cannot convert between incompatible pixmaps");
- }
+ if (copy_spots && ss != ds)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "incompatible number of spots when converting pixmap");
+ if (!da && sa)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "cannot drop alpha when converting pixmap");
if ((int)w < 0 || h < 0)
return;
@@ -1943,11 +1936,10 @@ static void fast_cmyk_to_rgb(fz_context *ctx, fz_pixmap *dst, fz_pixmap *src, fz
unsigned char r,g,b;
/* Spots must match, and we can never drop alpha (but we can invent it) */
- if ((copy_spots && ss != ds) || (!da && sa))
- {
- assert("This should never happen" == NULL);
- fz_throw(ctx, FZ_ERROR_GENERIC, "Cannot convert between incompatible pixmaps");
- }
+ if (copy_spots && ss != ds)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "incompatible number of spots when converting pixmap");
+ if (!da && sa)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "cannot drop alpha when converting pixmap");
if ((int)w < 0 || h < 0)
return;
@@ -2089,11 +2081,10 @@ static void fast_cmyk_to_bgr(fz_context *ctx, fz_pixmap *dst, fz_pixmap *src, fz
unsigned char r,g,b;
/* Spots must match, and we can never drop alpha (but we can invent it) */
- if ((copy_spots && ss != ds) || (!da && sa))
- {
- assert("This should never happen" == NULL);
- fz_throw(ctx, FZ_ERROR_GENERIC, "Cannot convert between incompatible pixmaps");
- }
+ if (copy_spots && ss != ds)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "incompatible number of spots when converting pixmap");
+ if (!da && sa)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "cannot drop alpha when converting pixmap");
if ((int)w < 0 || h < 0)
return;
@@ -2236,11 +2227,10 @@ static void fast_rgb_to_bgr(fz_context *ctx, fz_pixmap *dst, fz_pixmap *src, fz_
ptrdiff_t s_line_inc = src->stride - w * sn;
/* If copying spots, they must match, and we can never drop alpha (but we can invent it) */
- if ((copy_spots && ss != ds) || (!da && sa))
- {
- assert("This should never happen" == NULL);
- fz_throw(ctx, FZ_ERROR_GENERIC, "Cannot convert between incompatible pixmaps");
- }
+ if (copy_spots && ss != ds)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "incompatible number of spots when converting pixmap");
+ if (!da && sa)
+ fz_throw(ctx, FZ_ERROR_GENERIC, "cannot drop alpha when converting pixmap");
if ((int)w < 0 || h < 0)
return;