[ovs-dev] [PATCH v3] checkpatch.py: A simple script for finding patch issues

[Ben Pfaff]

On Thu, Mar 24, 2016 at 05:45:40PM -0400, Aaron Conole wrote:
> Most projects have a checkpatch facility, which can be used as a pre-commit
> sanity check. This introduces such a mechanism to the Open vSwitch project
> to catch some of the more silly formatting mistakes which can occur. It is
> not meant to replace good code review practices, but it can help eliminate
> the silly code review issues which get added.
> 
> Suggested-by: Mauricio Vásquez <mauricio.vasquezbernal at studenti.polito.it>
> Signed-off-by: Aaron Conole <aconole at redhat.com>
> ---
> v2:
>  - Moved to the utilities directory
>  - Fixed up according to flake8
>  - Added an ignore for leading tabs with .mk file modifications
>  - hooked up the return values
> 
> v3:
>  - Added to EXTRA_DIST
>  - Made it executable
>  - Support `git show | ./utilities/checkpatch.py`
>  - Added help text
>  - Added the ability to turn off most of the tests (except line-length)
>  - Amended CONTRIBUTING.md to also note to use linux checkpatch.pl for
>    datapath

This seems potentially useful, especially if it is enhanced over time to
add more tests.

It looks for various kinds of tags and counts them as 'extra_people',
but I don't see anything that actually uses that count.  Is there
something missing there?