Off-by-many error

[Boudewijn Rempt]

On Sunday 21 August 2005 16:21, Boudewijn Rempt wrote:

> The slower code works correctly, the clever code doesn't -- and I cannot
> find out what exaclyt doesn't work there, so I'll just checkin the correct
> code for now... 

I discovered what was wrong... It was not the code in KisPaintDevice; we used 
the wrong lcms type for cmyk. We used type_cmyk_8, which doesn't provide for 
the alpha byte at the end of the cmyk pixel (cmyka). So, converting one pixel 
always worked, but converting more than one pixel gave very amusing errors, 
because of the increasing off-ness -- which was reset on every tile boundary.

Anyway, it appears the right lcms type to use is type_cmyk5_8. This may give 
is grief in future, though, because with this type, lcms doesn't check 
whether the colorspace is correct for a given transform.

-- 
Boudewijn Rempt 
http://www.valdyas.org/fading/index.cgi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/kimageshop/attachments/20050822/efb7fb22/attachment.pgp