Subject: Re: CVS commit: src/sys
To: David Young <dyoung@netbsd.org>
From: Tom Spindler <dogcow@babymeat.com>
List: source-changes
Date: 11/16/2006 15:45:49
The "superfluous curly braces" removal bothers me a bit - in particular,
in ip_gre.c,

-       } else {
+       } else  
                IF_ENQUEUE(ifq, m);
-       }

Macros have a nasty habit of sometimes becoming multiple statements.
(Admittedly, in this case, since it's a do { ... } while (0), presumably
that behavior wouldn't ever happen.)