Re: [ovs-dev] [PATCH] checkpatch: Also check switch, HMAP_FOR_EACH, etc.

2017-05-31 Thread Ben Pfaff
On Tue, May 30, 2017 at 04:24:55PM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > > The switch statement and our FOR_EACH macro iteration constructs have the > > same rules as if, for, and while. > > > > Signed-off-by: Ben Pfaff > > --- > > At first, I was thinking that the FOR_EACH constru

Re: [ovs-dev] [PATCH] checkpatch: Also check switch, HMAP_FOR_EACH, etc.

2017-05-30 Thread Aaron Conole
Ben Pfaff writes: > The switch statement and our FOR_EACH macro iteration constructs have the > same rules as if, for, and while. > > Signed-off-by: Ben Pfaff > --- At first, I was thinking that the FOR_EACH constructs were treated separately, but actually it looks like there are just places wh

[ovs-dev] [PATCH] checkpatch: Also check switch, HMAP_FOR_EACH, etc.

2017-05-26 Thread Ben Pfaff
The switch statement and our FOR_EACH macro iteration constructs have the same rules as if, for, and while. Signed-off-by: Ben Pfaff --- utilities/checkpatch.py | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py