diff options
author | Philipp Oeser <info@graphics-engineer.com> | 2021-01-21 18:13:30 +0300 |
---|---|---|
committer | Philipp Oeser <info@graphics-engineer.com> | 2021-01-29 13:07:58 +0300 |
commit | 6f88053d852ad7f04b3d67b1905d88917f481ee3 (patch) | |
tree | f90f232bb27bb637fd198d4b7d3de27c599d9bbf /source/blender/imbuf | |
parent | 3fb5e838677d8db0c193da1a7c70cb5c392a4610 (diff) |
Fix T84661: read jpg pixel density
For jpeg, an image.resolution was always based on the default 72dpi, now
read the pixel density from the jpeg_decompress_struct, convert
according to unit and store in IMBuf's ppm.
Not 100% sure of all implications tbh., files I have checked seem to work
as expected now in the context of the report.
Maniphest Tasks: T84661
Differential Revision: https://developer.blender.org/D10166
Diffstat (limited to 'source/blender/imbuf')
-rw-r--r-- | source/blender/imbuf/intern/jpeg.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/source/blender/imbuf/intern/jpeg.c b/source/blender/imbuf/intern/jpeg.c index 7d4797def8f..440375f60dc 100644 --- a/source/blender/imbuf/intern/jpeg.c +++ b/source/blender/imbuf/intern/jpeg.c @@ -413,11 +413,22 @@ static ImBuf *ibJpegImageFromCinfo(struct jpeg_decompress_struct *cinfo, int fla jpeg_finish_decompress(cinfo); } - jpeg_destroy((j_common_ptr)cinfo); if (ibuf) { + /* Density_unit may be 0 for unknown, 1 for dots/inch, or 2 for dots/cm. */ + if (cinfo->density_unit == 1) { + /* Convert inches to meters. */ + ibuf->ppm[0] = cinfo->X_density / 0.0254f; + ibuf->ppm[1] = cinfo->Y_density / 0.0254f; + } + else if (cinfo->density_unit == 2) { + ibuf->ppm[0] = cinfo->X_density * 100.0f; + ibuf->ppm[1] = cinfo->Y_density * 100.0f; + } + ibuf->ftype = IMB_FTYPE_JPG; ibuf->foptions.quality = MIN2(ibuf_quality, 100); } + jpeg_destroy((j_common_ptr)cinfo); } return ibuf; |