From 8e25eef3d07432763cbd61315e0ec1df15213cf3 Mon Sep 17 00:00:00 2001 From: John Cupitt Date: Sun, 22 Nov 2020 16:00:34 +0000 Subject: [PATCH] make ppm load default to msb first We has lsb first as the default, breaking 16-bit PPM load. Thanks ewelot. see https://github.com/libvips/libvips/issues/1894 --- ChangeLog | 1 + libvips/foreign/ppmload.c | 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index e4af77e7..e1a27910 100644 --- a/ChangeLog +++ b/ChangeLog @@ -10,6 +10,7 @@ - better GraphicsMagick image write [bfriesen] - add missing read loops to spng, heif, giflib and ppm load [kleisauke] - block zero width or height images from imagemagick load [Koen1999] +- fix msb_first default in ppm load [ewelot] 6/9/20 started 8.10.2 - update magicksave/load profile handling [kelilevi] diff --git a/libvips/foreign/ppmload.c b/libvips/foreign/ppmload.c index 430a5572..7933c03b 100644 --- a/libvips/foreign/ppmload.c +++ b/libvips/foreign/ppmload.c @@ -39,6 +39,8 @@ * - ban max_vaue < 0 * 27/6/20 * - add ppmload_source + * 22/11/20 + * - fix msb_first default [ewelot] */ /* @@ -254,7 +256,7 @@ vips_foreign_load_ppm_parse_header( VipsForeignLoadPpm *ppm ) /* Default ... can be changed below for PFM images. */ - ppm->msb_first = 0; + ppm->msb_first = 1; /* Read in size. */