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

List:       kde-commits
Subject:    KDE/kdegraphics/kolourpaint/pixmapfx
From:       Clarence Dang <dang () kde ! org>
Date:       2007-10-22 2:44:47
Message-ID: 1193021087.603616.20621.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 727900 by dang:

-debug but always show the screen depth diagnostic line, even in release mode \
(crucially important for diagnostic purposes, given the XRENDER hack we invoke at \
32-bit).

Restore a debug line that was nuked in a previous commit.


 M  +6 -3      kpPixmapFX_MaskOps.cpp  


--- trunk/KDE/kdegraphics/kolourpaint/pixmapfx/kpPixmapFX_MaskOps.cpp #727899:727900
@@ -26,7 +26,7 @@
 */
 
 
-#define DEBUG_KP_PIXMAP_FX 1
+#define DEBUG_KP_PIXMAP_FX 0
 
 
 #include <kpPixmapFX.h>
@@ -115,6 +115,7 @@
         KApplication app;
 
         const int depth = QPixmap::defaultDepth ();
+        KP_PRINTF (("Child: writing default depth\n"));
         write (fds [Write], &depth, sizeof (depth));
         KP_PRINTF (("Child: wrote default depth\n"));
 
@@ -153,9 +154,11 @@
 {
 #ifdef Q_WS_X11
     const int defaultDepth = QPixmapCalculateDefaultDepthWithoutQApplication ();
-    KP_PRINTF ("kpPixmapFX::initMaskOpsPre() QPixmap::defaultDepth=%d\n",
-               defaultDepth);
 
+    // This is important for diagnosing KolourPaint bugs so always print it
+    // -- even in release mode.
+    printf ("Starting KolourPaint on a %d-bit screen...\n", defaultDepth);
+
     if (defaultDepth == 32)
     {
         KP_PRINTF ("\tCannot handle alpha channel - disabling XRENDER\n");


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