So I guess our choices are: 1. Reinstate (void) only where the warning occurs. 2. Reinstate (void) everywhere return values aren't checked. What a nightmare. 3. Change defines such as the above to static inline functions. Disadvantage: The inline keyword is not in the C89 standard.
I have ran the build tests and do not know of any other cases like this. Certainly, a few more might show up but it is not an endemic problem at the moment.
At its worst, it is only a warning. Greg