aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--fiv-io.c24
1 files changed, 13 insertions, 11 deletions
diff --git a/fiv-io.c b/fiv-io.c
index 6722019..67805e9 100644
--- a/fiv-io.c
+++ b/fiv-io.c
@@ -2298,8 +2298,8 @@ load_resvg_destroy(void *closure)
}
static cairo_surface_t *
-load_resvg_render_internal(
- FivIoRenderClosureResvg *self, double scale, GError **error)
+load_resvg_render_internal(FivIoRenderClosureResvg *self,
+ double scale, FivIoProfile target, GError **error)
{
double w = ceil(self->width * scale), h = ceil(self->height * scale);
if (w > SHRT_MAX || h > SHRT_MAX) {
@@ -2324,21 +2324,21 @@ load_resvg_render_internal(
cairo_image_surface_get_width(surface),
cairo_image_surface_get_height(surface), (char *) pixels);
- // TODO(p): Also apply colour management, we'll need to un-premultiply.
for (int i = 0; i < w * h; i++) {
uint32_t rgba = g_ntohl(pixels[i]);
pixels[i] = rgba << 24 | rgba >> 8;
}
cairo_surface_mark_dirty(surface);
- return surface;
+ return fiv_io_profile_finalize(surface, target);
}
static cairo_surface_t *
load_resvg_render(FivIoRenderClosure *closure, double scale)
{
FivIoRenderClosureResvg *self = (FivIoRenderClosureResvg *) closure;
- return load_resvg_render_internal(self, scale, NULL);
+ // TODO(p): Somehow get the target colour management profile.
+ return load_resvg_render_internal(self, scale, NULL, NULL);
}
static const char *
@@ -2395,7 +2395,8 @@ open_resvg(
closure->width = size.width;
closure->height = size.height;
- cairo_surface_t *surface = load_resvg_render_internal(closure, 1., error);
+ cairo_surface_t *surface =
+ load_resvg_render_internal(closure, 1., ctx->screen_profile, error);
if (!surface) {
load_resvg_destroy(closure);
return NULL;
@@ -2450,6 +2451,8 @@ load_librsvg_render(FivIoRenderClosure *closure, double scale)
cairo_surface_destroy(surface);
return NULL;
}
+
+ // TODO(p): Somehow get the target colour management profile.
return surface;
}
@@ -2515,7 +2518,7 @@ open_librsvg(
closure->height = h;
cairo_surface_set_user_data(
surface, &fiv_io_key_render, closure, load_librsvg_destroy);
- return surface;
+ return fiv_io_profile_finalize(surface, ctx->screen_profile);
}
#endif // HAVE_LIBRSVG --------------------------------------------------------
@@ -2998,6 +3001,9 @@ load_libtiff_directory(TIFF *tiff, GError **error)
cairo_surface_mark_dirty(surface);
// XXX: The whole file is essentially an Exif, any ideas?
+ // TODO(p): TIFF has a number of fields that an ICC profile can be
+ // constructed from--it's not a good idea to blindly default to sRGB
+ // if we don't find an ICC profile.
const uint32_t meta_len = 0;
const void *meta = NULL;
if (TIFFGetField(tiff, TIFFTAG_ICCPROFILE, &meta_len, &meta)) {
@@ -3073,10 +3079,6 @@ fail:
TIFFSetWarningHandlerExt(whe);
TIFFSetErrorHandler(eh);
TIFFSetWarningHandler(wh);
-
- // TODO(p): Colour management even for un/associated alpha channels.
- // Note that TIFF has a number of fields that an ICC profile can be
- // constructed from--it's not a good idea to blindly assume sRGB.
return fiv_io_profile_finalize(result, ctx->screen_profile);
}