[cvs] bogofilter/src token.c,1.112,1.113

[Matthias Andree]

David Relson <relson at osagesoftware.com> writes:

>> -#define WCLEAR(n)	n.leng = 0
>> -#define WFREE(n)	word_free(n); n = NULL
>> +#define WFREE(n)	do { word_free((n)); (n) = NULL; } while(0)
>>  
>>  /* Function Prototypes */
>
> Safety is not called for here.  What does "(n) = NULL" mean???

Nothing else than n = NULL in the regular case.

> If "n" is not a simple variable name, getting a compiler error is the
> _right_ thing.

If that's what you're after, you want something that supports call by
reference and type-checked inline functions. That would mean going
beyond C89.

-- 
Matthias Andree
_______________________________________________
Bogofilter-dev mailing list
Bogofilter-dev at bogofilter.org
http://www.bogofilter.org/mailman/listinfo/bogofilter-dev