🚀 go-pugleaf

Antony Pace (UK/CH) wrote:
> 
> Can anyone help?
> 
> Here is my first attempt at the macro.
> 
> #define CLEAR_ERROR(e) { \
>   (e).a = 0;             \
>   (e).b = 0;             \
>   (e).c = 0;             \
>   TRACE("clear error");  \
> }
> 
> In this code segment, my macro does not work:
> 
>   ...
>   if (a==3)
>     CLEAR_ERROR(error);
>   else
>     b=5;

Well, I'm not much of a theoretician, so perhaps my answer won't count
for much.

Anyway, if I were you, I'd publish that the call of the CLEAR_ERROR(e)
macro (especially as an if/else clause) should either

    a)  always be within braces -- { CLEAR_ERROR (e) }
or
    b)  never be followed by a semicolon --  CLEAR_ERROR (e)

My personal style choice would be 'a'.  I use that style so that I can
always expand an if/else clause without forgetting to add a brace.  I
just plain don't care that it may add extra white space.

Just my 2c.
-- 

Roger Ramhoff
It *can* be better

Field Consultant			William.R.Ramhoff@cdc.com
Control Data Systems			(810) 262-8235