[webkit-reviews] review granted: [Bug 12851] improve gdk's notImplemented() macro : [Attachment 13309] improve gdk's notImplemented() macro

bugzilla-request-daemon at macosforge.org bugzilla-request-daemon at macosforge.org
Wed Feb 21 22:41:44 PST 2007

Adam Roben <aroben at apple.com> has granted Adam Roben <aroben at apple.com>'s
request for review:
Bug 12851: improve gdk's notImplemented() macro

Attachment 13309: improve gdk's notImplemented() macro

------- Additional Comments from Adam Roben <aroben at apple.com>
+#define notImplementedGdk() do { \
+    static int count; \
+    if (!count && !getenv("DISABLE_NI_WARNING")) \
+	 fprintf(stderr, "FIXME: UNIMPLEMENTED %s %s:%d\n",
+    ++count; \
+ } while(0)

   Might as well just use a "bool printed" instead of "int count".

   Instead of putting this in config.h, you should probably just create your
own header platform/gdk/NotImplemented.h that only contains this macro.

   You might also want to uncomment any other notImplemented() calls that were
commented out for being too chatty.


More information about the webkit-reviews mailing list