Legacy ImageMagick Discussions Archive

Merci beaucoup

Hello,

the patch has been applied into the trunk, but the (automatic?) merge into branch 6.7.2 has failed :

if you compare
http://trac.imagemagick.org/browser/ImageMagick/branches/ImageMagick-6.7.2/coders/jpeg.c#L646
and
http://trac.imagemagick.org/browser/ImageMagick/trunk/coders/jpeg.c#L671 ...

First of all, if someone should be thanked, it's you !

But I still don't understand why you copy a buffer twice at the same location (profile->datum). The first argument of memcpy() is the destination pointer, not the source one.

Would you rather write, in http://trac.imagemagick.org/browser ...

Hello,

The ReadProfile() function in coders/jpeg.c deals with a possible previous profile in the image:

previous_profile=GetImageProfile(image,name);
if (previous_profile != (const StringInfo *) NULL)
{
SetStringInfoLength(profile,GetStringInfoLength(profile)+
GetStringInfoLength(previous ...