I'd kinda actually rather see it go the other way, #if (ADVANCED_TIMERS) without the check on it being defined.  This was to avoid the surprise of doing #define ADVANCED_TIMERS 0, and seeing that it included it just the same as #define ADVANCED_TIMERS 1