On Mon, Apr 28, 2014 at 10:26:04AM +0200, Marc Marí wrote: > /* Debug EEPRO100 card. */ > #if 0 > -# define DEBUG_EEPRO100 > +#define DEBUG_EEPRO100 1 > #endif
There seem to be a few ways of doing this (commenting out the line, putting it in #if 0, etc). Feel free to get rid of #define DEBUG_foo 1 lines completely, leaving just the #ifndef (I think Kevin suggested this). Editing the file is bad since it's easy to forget to remove changes when doing git-commit(1). Debugging should be enabled by recompiling instead of editing source files: $ ./configure --extra-cflags=-DDEBUG_FOO=1 && make But this is just a personal preference, if you want to leave the lines it's fine. Stefan
