[gdal-dev] MrSID Color Spaces

[Martin Chapman]

Even,

 

That's correct, I did not actually build it.  Our third party libraries are
built by another person on the team and it doesn't happen quick.  I thought
he was asking me whether it looks correct.  I apologize for the
mis-communication.

 

Best regards,

Martin

 

From: Even Rouault [mailto:even.rouault at spatialys.com] 
Sent: Wednesday, May 17, 2017 9:31 AM
To: Martin Chapman
Cc: 'SIMON Nicolas'; gdal-dev at lists.osgeo.org
Subject: Re: [gdal-dev] MrSID Color Spaces

 

On mercredi 17 mai 2017 09:10:42 CEST Martin Chapman wrote:

> It's an enum value defined in at least version 8.0 and greater. Maybe in

> earlier versions but I am using version 9.1.

> 

 

Indeed, that's an *enum value*, not a #define !

 

typedef enum LTIColorSpace

{

[...]

LTI_COLORSPACE_GRAYSCALEA = LTI_COLORSPACE_GRAYSCALE |
LTI_COLORSPACE_MASK_ALPHA,

LTI_COLORSPACE_GRAYSCALEA_PM = LTI_COLORSPACE_GRAYSCALEA |
LTI_COLORSPACE_MASK_ALPHA_PRE_MULTIPLIED,

 

 

I interpreted your "Looks good" as "applied and successfully tested"...

 

I do have SDK 8 indeed, so I replaced the conditional test with:

 

#if defined(LTI_SDK_MAJOR) && LTI_SDK_MAJOR >= 8

 

-- 

Spatialys - Geospatial professional services

http://www.spatialys.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/gdal-dev/attachments/20170517/e8deadf7/attachment.html>