Holger Hans Peter Freyther <holger.freyther@trolltech.com>
authorhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 21 Apr 2008 11:15:34 +0000 (11:15 +0000)
committerhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 21 Apr 2008 11:15:34 +0000 (11:15 +0000)
* Try to bring the notImplemented implementations closer together. This will
  print each unimplemted method name only once.

* We might want to change wtf/Assertions.cpp to use qDebug

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@32303 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/platform/NotImplemented.h

index a3f33d5..755e01b 100644 (file)
@@ -2,6 +2,17 @@
 
         Reviewed by Simon.
 
+        * Try to bring the notImplemented implementations closer together. This will
+        print each unimplemted method name only once.
+
+        * We might want to change wtf/Assertions.cpp to use qDebug
+
+        * platform/NotImplemented.h:
+
+2008-04-21  Holger Hans Peter Freyther  <holger.freyther@trolltech.com>
+
+        Reviewed by Simon.
+
         * Remove the special Qt ctor of BitmapImage as it did not work. The Bitmap returned
         0 as width and after intialising drawing would fail. BitmapImage relies on a proper
         ImageSource.
index f942065..dd815ab 100644 (file)
@@ -31,6 +31,8 @@
 
 #if PLATFORM(GTK)
     #define supressNotImplementedWarning() getenv("DISABLE_NI_WARNING")
+#elif PLATFORM(QT)
+    #define supressNotImplementedWarning() !qgetenv("DISABLE_NI_WARNING").isEmpty()
 #else
     #define supressNotImplementedWarning() false
 #endif
 #elif PLATFORM(QT)
     #include <qglobal.h>
     #include <qbytearray.h>
-    #define notImplemented() \
-        if (qgetenv("DISABLE_NI_WARNING").isEmpty()) \
-            qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, WTF_PRETTY_FUNCTION)
+    #define notImplemented() do { \
+        static bool havePrinted = false; \
+        if (!havePrinted && !supressNotImplementedWarning()) { \
+            qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, WTF_PRETTY_FUNCTION); \
+            havePrinted = true; \
+        } \
+    } while (0)
 
 #else