https://bugs.kde.org/show_bug.cgi?id=281761 -- scaling between channel sizes

Boudewijn Rempt boud at valdyas.org
Mon Sep 19 08:22:36 UTC 2011


On Monday 19 September 2011 Sep, Boudewijn Rempt wrote:
> On Sunday 18 September 2011 Sep, Silvio Heinrich wrote:
> > On 09/18/2011 03:06 PM, Boudewijn Rempt wrote:
> > > hi,
> > >
> > > I'm stuck with this bug... Could someone with adequate templating foo please check my patch and help me out a bit? I get a crash now if I follow the instructions in the bug report exactly.
> > 
> > Ok I got it. Now it seems to work.
> > Now just one part is missing.
> > We need to test if the destination colorspace uses floating point 
> > channels so that the second part of the "else" part is executed.
> 
> Looks good! I think we can skip the check for floating point, though, since KoColorSpaceAbstract isn't used as a base class for floating point colorspaces, we use ctlcs for that. Otherwise, it would be simple enough -- we can check for the type of the first channel, since KoColorSpaceAbstract is guaranteed to be homogenous.
> 
> 

Here's my version with the check for floating point. For the rest it works perfectly, I just tested it.


-- 
Boudewijn Rempt
http://www.valdyas.org, http://www.krita.org, http://www.boudewijnrempt.nl
-------------- next part --------------
A non-text attachment was scrubbed...
Name: convert.patch
Type: text/x-patch
Size: 4320 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kimageshop/attachments/20110919/9ed0faee/attachment.patch>


More information about the kimageshop mailing list