[riot-notifications] [RIOT-OS/RIOT] core: debug: remove parenthesis around ENABLE_DEBUG define (#15365)
notifications at github.com
Tue Nov 3 12:19:24 CET 2020
> > This allows for ENABLE_DEBUG to not be defined before including debug.h when the same behavior as #define ENABLE_DEBUG 0 is desired.
> > Removing a `#define ENABLE_DEBUG 0`, but not the `debug.h` should now be compilable.
> I think it was like that due to this:
Not really sure what you mean... Please rephrase.
> > why introduce the extra dependency when the goal (compile time checking) is met already?
> > This is not replacing `#if DEFINE` with `if(DEFINE)`, it was already the latter, and the header made sure that ENABLE_DEBUG is always set.
Removed the `IS_ACTIVE()` changes, and thus this PR only removes the parenthesis around the `0` in the define.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications