[Digikam-devel] Upgrading libjpeg (6b-6 -> 7-1) problems

[Gilles Caulier]

2009/7/18 Mikolaj Machowski <mikmach at wp.pl>:
> On Saturday 18 July 2009 11:28:01 Andi Clemens wrote:
>> Hi,
>>
>> I'm right now analyzing if this problem is KDElibs specific or some digiKam
>> issue.
>> I'm pretty sure the "OpenWith" context menu entry worked before with jpeg
>> files, but now it doesn't.
>> I installed digiKam 0.10 and it is grayed out, too. So far so good, sounds
>> like a problem with the new libjpeg.
>> But dolphin for example just works fine.
>>
>> Anyone with libjpeg v7 can confirm this?
>> I will try to get an older version of the lib again, if it is working, I
>> need to check dolphin again.
>
> Not sure if/how related but just got this mail from KHTML commitfilter:
>
> KDE/kdelibs/khtml/imload/decoders
> From: Maks Orlovich <maksim at kde.org>
>  To: kde-commits at kde.org
>  CC: khtml-cvs at kde.org
>  Date: 2009-07-18 00:31
>
> SVN commit 998545 by orlovich:
>
> Make this work with libjpeg7 as well. Credit to Daniel Winter
> for providing me the information to fix this and testing the fix.
>
> BUG: 198779
>
>
>  M  +1 -1      jpegloader.cpp
>
>
> --- trunk/KDE/kdelibs/khtml/imload/decoders/jpegloader.cpp #998544:998545
> @@ -324,7 +324,7 @@
>                 scaleDown *= 2;
>             }
>
> -            cinfo.scale_denom = scaleDown;
> +            cinfo.scale_denom *= scaleDown;
>
>             if (scaleDown > 8) {
>                 // Still didn't fit... Abort.
> _______________________________________________
> Digikam-devel mailing list
> Digikam-devel at kde.org
> https://mail.kde.org/mailman/listinfo/digikam-devel
>

Oh, nice mik. I'm not sure if we use this libjpeg parameter, but at
least we know a solution

Gilles