I would have said do {} while (0).

You've been suggesting this for a number of reviews, what's the point if you don't mind be asking ?

I believe most the ones I suggested are void functions, no return value; @hselasky in the past has always suggested to me for LinuxKPI to use do {} while(0) in those cases and that seems general style in FreeBSD.

Here's a quick test putting a (0); into an empty main function and you get a warning ...
I think we simply suppress them in the builds currently:

share/mk/bsd.sys.mk:CWARNFLAGS.clang+=  -Wno-tautological-compare -Wno-unused-value\
share/mk/bsd.sys.mk:            -Wno-error=unused-value

% cc -Wall 0.c -o 0-2
0.c:7:3: warning: expression result unused [-Wunused-value]
        (0);
         ^
1 warning generated.

Right, do { ... } while(0) is generally good practice so that it behaves similarly to a function with respect to cases like @bz mentions, the trailing ; after the macro behaves as expected, and (in the case of macros with actual bodies) it works properly inside of another block like if ... else ....

I think it's reasonable to just always put void function-like macros in do { } while (0)

Thanks for the explanation, I'll update all the reviews to do this :)