averello/ImageMagick-mirror
1
0
Fork 0
mirror of https://github.com/ImageMagick/ImageMagick.git synced 2026-05-25 11:24:54 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fix JXL orientation when writing. (#8673)

Browse Source 
JXL documentation told us to ignore EXIF because orientation is
encoded in the stream itself.
ImageMagick was keeping the EXIF but not setting the stream.
Both should be kept and synchronized.
This commit is contained in:
Nicolas Cornu
2026-04-17 18:12:18 +02:00
committed by GitHub
parent 21fc4a99b0
commit b074bd9ce7
1 changed files with 26 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
coders/jxl.c
+26
View File
@@ -191,6 +191,31 @@ static inline StorageType JXLDataTypeToStorageType(Image *image,
}
}
static inline JxlOrientation OrientationToJXLOrientation(
const OrientationType orientation)
{
switch (orientation)
{
default:
case TopLeftOrientation:
return(JXL_ORIENT_IDENTITY);
case TopRightOrientation:
return(JXL_ORIENT_FLIP_HORIZONTAL);
case BottomRightOrientation:
return(JXL_ORIENT_ROTATE_180);
case BottomLeftOrientation:
return(JXL_ORIENT_FLIP_VERTICAL);
case LeftTopOrientation:
return(JXL_ORIENT_TRANSPOSE);
case RightTopOrientation:
return(JXL_ORIENT_ROTATE_90_CW);
case RightBottomOrientation:
return(JXL_ORIENT_ANTI_TRANSPOSE);
case LeftBottomOrientation:
return(JXL_ORIENT_ROTATE_90_CCW);
}
}
static inline OrientationType JXLOrientationToOrientation(
const JxlOrientation orientation)
{
@@ -1111,6 +1136,7 @@ static MagickBooleanType WriteJXLImage(const ImageInfo *image_info,Image *image,
basic_info.animation.tps_denominator=1;
JxlEncoderInitFrameHeader(&frame_header);
}
basic_info.orientation=OrientationToJXLOrientation(image->orientation);
jxl_status=JxlEncoderSetBasicInfo(jxl_info,&basic_info);
if (jxl_status != JXL_ENC_SUCCESS)
{