[ovs-dev] [PATCH] checkpatch: Check FOR_EACH loops with numbers.
blp at ovn.org
Fri Jul 12 15:49:55 UTC 2019
On Fri, Jul 12, 2019 at 06:33:32PM +0300, Ilya Maximets wrote:
> On 12.07.2019 18:20, Ben Pfaff wrote:
> > On Fri, Jul 12, 2019 at 04:04:38PM +0300, Ilya Maximets wrote:
> >> OVS has defines for loops like 'BITMAP_FOR_EACH_1' or
> >> 'ULLONG_FOR_EACH_1', but the regexp in checkpatch doesn't match with
> >> numbers and skips these loops while checking.
> >> This patch adds numbers into regexp and adds some FOR_EACH loops to
> >> the unit tests.
> >> Signed-off-by: Ilya Maximets <i.maximets at samsung.com>
> > Acked-by: Ben Pfaff <blp at ovn.org>
> Thanks, Ben.
> Can we consider this as a "bug fix" and apply or should we wait until
> "freeze" finished?
I'd go ahead with it. checkpatch can't break any real code.
More information about the dev