Legacy ImageMagick Discussions Archive

Hi, Here are a few proposed tweaks to IM core based on my own internal modifications: - CORE_png / pngvalid.c: interferes with the running of Boost test since it defines an int main(). Recommend removing 'pngvalid.c' from the default build. OpenCV has already removed the file from their build. Refer...

Thank you! One more submission for you; this reads the extended file format to determine lossy or lossless based on the chunk data. Could probably use another set of eyes to ensure this won't crash on edge cases // from libwebp, webp.c static inline uint32_t get_le24(const unsigned char* const data)...

Hi, thanks for getting this integrated! I was reviewing the latest changes to webp prior to downloading/compiling, and I noticed an issue that might need attention: http://trac.imagemagick.org/browser/ImageMagick/branches/ImageMagick-6/coders/webp.c Line 176: if ((stream[15] == 'L') || (stream[15] =...

I just started working with WEBP and noticed that the coder can't do WEBP lossless, and it also has a memory leak. I made some significant changes to core_coders/webp.c which allow IM users to define codec-specific optimizations and, most notably, lossless encoding. Below is my code for your conside...

Thank you!

Reference post: http://www.imagemagick.org/discourse-server/viewtopic.php?f=1&t=20836 Download the Webp library and use the cwebp and dwebp tools to convert your PNG to the Webp format. With these tools we're getting a black alpha channel. If you can get them to produce transparent Webp, let us ...

Awesome, thank you!

Hi, Magick++ doesn't seem to write multi-image Images to a single blob. Regardless if the image has multiple sub-images, it always calls ImageToBlob, when it appears it should be calling ImagesToBlob instead, if needed. I'm guessing this applies to multi-page TIFFs, PDFs, and others as well? Anyhoo,...

Fixed/confirmed in IM 6.8.2.2. Thank you!

Thanks for your response regarding the transparent pixel issue. Is the grayscale image issue still open then?

Glenn, I was looking through the changelog for IM 6.8.2.x. I noted that you have made changes related to PNG en/decoding that may have addressed these issues, but I can't be certain. Were both of these issues addressed? The first issue being the grayscale image, the second being the transparent pixe...

As a follow up, I stumbled on a workaround while doing some grayscale image processing. Basically, I manually set the colorspace to RGB and then call IsGrayImage in attribute.c, which contains auto-detect logic based on image pixel data. Also, I noticed that any transparent, non-gray pixels will cau...

Glenn, thanks for the response. I'm doing image manipulation through code; I just provided a simple command line to illustrate the problem. Moreover, I wouldn't necessarily know if the input JPEG was progressive or not, so I couldn't just hard-code in a static value. I've already come up with a work...

Bigger-picture issues aside, I'd recommend the following changes to solve the interlace problem: image.c, line 1281: from: image_info->interlace=NoInterlace; to: image_info->interlace=UndefinedInterlace; jpeg.c, line 2172: from: if ((LocaleCompare(image_info->magick,"PJPEG") == 0) || (imag...

Version: 6.8.0.2 At first, there was an issue that caught my eye. I am using magick++, but am able to recreate an example of the issue with this command: convert.exe progressive.jpg nonprogressive.jpg This turns a progressive JPEG into a non-progressive JPEG. Verified before and after with identify....