Ben Pfaff <[email protected]> writes: > On Wed, Apr 19, 2017 at 05:42:58PM -0400, Aaron Conole wrote: >> Lance pointed to a problem where scripts were incorrectly being flagged as >> pointer spacing warnings. As an example, the text: >> >> --u*|-u) >> >> would flag the warning. >> >> Since the type name should always have a space in front of it, change the >> regex to require some spacing. Additionally, ** is a common notation in >> comments to mean 'raise to the power of', so ensure that it is not >> accidentally flagged as well by adding a not-group populated with *. >> >> Reported-by: Lance Richardson <[email protected]> >> Signed-off-by: Aaron Conole <[email protected]> > > Should checkpatch limit C code warnings to C source files and header > files?
That's a good suggestion. I'll cook up a patch to whitelist the files that are definitely C-code: .c, .h, .c.in, .h.in as suffixes. You can hold off on applying this if you want to wait for that change as well - just let me know and I'll submit a series. Thanks, -Aaron _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
