[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdegraphics/kolourpaint
From:       Aaron J. Seigo <aseigo () kde ! org>
Date:       2007-10-28 23:21:16
Message-ID: 1193613676.287136.25824.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 730500 by aseigo:

proper fix, as per Clarence's directions (who is out of range of svn atm =)

CCMAIL:dang@kde.org


 M  +0 -1      imagelib/effects/kpEffectReduceColors.cpp  
 M  +1 -1      pixmapfx/kpPixmapFX_Transforms.cpp  


--- trunk/KDE/kdegraphics/kolourpaint/imagelib/effects/kpEffectReduceColors.cpp #730499:730500
@@ -53,7 +53,6 @@
         return QImage::Format_Indexed8;
 
     // 24-bit is known as 32-bit with QImage.
-    case 24:  // TODO: Why is this needed for flip on a 24-bit screen?
     case 32:
         // TODO: If the src image has no mask, we should really return RGB32, to
         //       avoid introducing a mask.
--- trunk/KDE/kdegraphics/kolourpaint/pixmapfx/kpPixmapFX_Transforms.cpp #730499:730500
@@ -817,7 +817,7 @@
         // (and therefore, <pm>) is only 8-bit.  The name "ReduceColors"
         // is a misnomer in this case.
         newQImage = kpEffectReduceColors::convertImageDepth (
-            newQImage, 24/*new depth*/, false/*no dither*/);
+            newQImage, 32/*new depth*/, false/*no dither*/);
 
         // For depth>1, we could do the entire transform using ::TransformPixmap()
         // but given the rounding error hacks in that method, we'd rather use
[prev in list] [next in list] [prev in thread] [next in thread]