[Lcd4linux-devel] Little Patch for debug.h

[Ernst B. <e.b...@xe...>]

Hi list,

A little one-line patch to debug.h:

-------------------- snip ---------------
 extern int running_background;
 extern int verbose_level;

-void message(const int level, const char *format, ...);
+void message(const int level, const char *format, ...) __attribute__((format 
(__printf__, 2, 3) ));

 #define debug(args...) message (2, __FILE__ ": " args)
 #define info(args...)  message (1, args)
----------------- snip -------------------

It tells gcc that "message" takes a printf-like format string, so gcc can 
check if the varargs match.
Saved me a lot of bug-hunting time for spurious segfaults caused by wrong 
debug statements.

Should work with gcc 3.3.1 or later, probably earlier versions of gcc, too.

One could add some gcc version-check #ifdefs arround it, but since other parts 
of the code already use __attribute__(( unused )) and the like, its probably 
not neccessary.

/Ernst