Preprozessor #ifdef um eine Funktion ab zu schalten

[David H. (davidm)]

// Debugout funktion verwenden
#define SF_DEBUGOUT(text,flash_addr,stats)    event_buffer_print(text,flash_addr,stats);

// Debugout nicht verwenden (zuminest denke ich das es so geht)
#define SF_DEBUGOUT  (void) 0;

Jetzt möchte ich die untere funktion abschalten, jedoch geht das nicht 
da
SF_DEBUGOUT ja leider definiert ist (mit void 0), gibt es hier schicke 
wege das irgendwie doch zu tun?

#ifdef SF_DEBUGOUT   
event_buffer_print(text,flash_addr,stats) {

//.... stuff

}
#endif

[Frank W. (wesoft)]

Wie wäre es mit

#undef SF_DEBUGOUT

Frank

[def (Gast)]

statt:

#ifdef SF_DEBUGOUT
...

so:

#if SF_DEBUGOUT
...

mit:

#define SF_DEBUGOUT 0

wird Block übersprungen

mit:

#define SF_DEBUGOUT 1  // eigentlich >0

wird Block übersetzt

[Oliver (Gast)]

#ifdef DEBUG
// Debugout funktion verwenden
#define SF_DEBUGOUT(text,flash_addr,stats)    event_buffer_print(text,flash_addr,stats);
#else
// Debugout nicht verwenden (zuminest denke ich das es so geht)
#define SF_DEBUGOUT  (void) 0;
#endif

[Jörg W. (dl8dtl) (Moderator)]

#ifdef DEBUG
// Debugout funktion verwenden
#define SF_DEBUGOUT(text,flash_addr,stats)    event_buffer_print(text,flash_addr,stats) // *ohne* Semikolon!
#else
// Debugout nicht verwenden (zuminest denke ich das es so geht)
#define SF_DEBUGOUT(text,flash_addr,stats) // einfach nichts hier
#endif

[David H. (davidm)]

Vielen Dank für die Tips.

btw, das muss scheinbar so aussehen, zumindest macht microchip das in 
ihrer appio.h :-)

#define SF_DEBUGOUT(ignore,ignore2,ignore3)  ((void) 0)

eh, wir haben aber nicht aneinander vorbei geredet?

Was ich da Plane ist das er die sachen nicht mehr mit ein compiled, 
sprich die debug funktionen komplett raus nimmt.

[Karl H. (kbuchegg)]

David Mueller schrieb:
> Vielen Dank für die Tips.
>
> btw, das muss scheinbar so aussehen, zumindest macht microchip das in
> ihrer appio.h :-)
>
>

> #define SF_DEBUGOUT(ignore,ignore2,ignore3)  ((void) 0)
>

>
>
> eh, wir haben aber nicht aneinander vorbei geredet?

Doch, ich denke schon.

Dir geht es darum, wie du mit diesem #define dann die eigentliche 
Funktion aus dem Compilat auch noch entfernen kannst.

Zu diesem Zwecke mach dir klar, dass es einen Unterschied zwischen

#ifdef

und

#if

gibt.
Ersteres fragt lediglich ab, ob es ein Makro gibt.
Letzteres kannst du zb verwenden um auch noch den Wert eines Makros 
auszuwerten

#if SF_DEBUGOUT == 0
...
#endif

Genauer gesagt, gilt für #if dasselbe was auch für ein normales C-if 
gilt: Es erwartet einen Ausdruck und der kann 0 oder nicht 0 sein. Je 
nachdem wird dann vom #if der nachfolgende Textblock mitgenommen oder 
nicht.

[Jörg W. (dl8dtl) (Moderator)]

David Mueller schrieb:
> das muss scheinbar so aussehen

Nein, einfach leer lassen sollte genauso gehen.  Der Unterschied ist
nur, dass das „((void) 0)“ syntaktisch eine Anweisung ist (welche
keine Wirkung hat), während beim Leerlassen eine leere Anweisung
entsteht (durch das nachfolgende Semikolon, welches man normalerweise
hinter dem Makroaufruf noch schreibt).

Eine Warnung kann ein Compiler für beide Fälle natürlich generieren,
oder eben auch nicht.  Da wir hier im GCC-Forum sind: mir ist nicht
bekannt, dass GCC eine leere Anweisung warnen würde.

[avr3 (Gast)]

Das geht z.B. so:


#ifdef DEBUG
  #define debug_printf  printf
#else
  #define debug_printf(...)
#endf

[avr3 (Gast)]

oder in deinem Fall:

#ifdef DEBUG
  #define SF_DEBUGOUT(a,b,c) event_buffer_print(a,b,c)
#else
  #define SF_DEBUGOUT(...)
#endif

[Dani Zwirner (Gast)]

Ist ein bischen spitzfindig, aber das ((void)0) hat auch seine Vorteile:

Mit

#ifdef DEBUG 
  #define A(x) printf("%i\n", x)
#else
  #define A(x) (void(0))
#endif

lässt sich das folgende Statement auch kompilieren wenn DEBUG nicht 
gesetzt ist:

for(int i=0;i<3;i++, A(i))
{
}

Mit

#define A(x)

würde der Kompiler einen Fehler melden.

[Jörg W. (dl8dtl) (Moderator)]

Dani Zwirner schrieb:
> würde der Kompiler einen Fehler melden.

Was ja auch nicht schlimm wäre.

Der kanonische Weg, um den Inhalt eines Makros sich so benehmen zu
lassen wie eine Funktion ist:

#define A(x) do {} while (0)

Würde ich aber für den Debug-Kram nur machen, wenn's unbedingt sein
muss.  Normalerweise stehen solche debug-Print-Anweisungen nur mehr
mutterseelenallein auf einer Zeile:

   DPRINTF("foo failed: %d\n", foo);