ENABLE_DEBUG per module?

Oleg_Hahm · 21 April 2015 08:14

Dear reflecting IOTlers,

some time ago Martine proposed to move the `ENABLE_DEBUG` macro from file to module context [1]. Since this would effect the whole RIOT printf debugging, I would like to call on the mailing list if someone thinks that this would be a bad idea.

Cheers, Oleg

[1] https://github.com/RIOT-OS/RIOT/pull/2632

Kaspar · 21 April 2015 09:26

Hey,

miri64 · 21 April 2015 09:53

Hi, just to add my rational to the discussion: I don’t like that I have to edit files to activate debugging messages (and potentially adding the change to my PR by accident). My PR eliminates - but does not prohibit (one can still add the #define ENABLE_DEBUG (1) before the debug.h include) - the need for that.

Cheers, Martine

Kaspar · 21 April 2015 11:29

Hey,

miri64 · 21 April 2015 11:36

Hi,

Kaspar · 21 April 2015 11:52

Hi,

Ludwig_Knupfer1 · 21 April 2015 18:02

You still can enable debugging on a per file base.

Cheers, Ludwig