]> git.proxmox.com Git - mirror_qemu.git/commit - scripts/checkpatch.pl
checkpatch: default to success if only warnings
authorPaolo Bonzini <pbonzini@redhat.com>
Tue, 9 Aug 2016 15:47:44 +0000 (17:47 +0200)
committerPaolo Bonzini <pbonzini@redhat.com>
Wed, 10 Aug 2016 10:44:51 +0000 (12:44 +0200)
commit141de8865488189ad9d75408b3e0ad24c6fff2bb
treebffcf2db9cc4eb2ae49576f307d3ade6203b6c2d
parentc2df8783251e16279229520f7cf5cb81c188a934
checkpatch: default to success if only warnings

CHK-level checks have been removed from checkpatch or bumped to
errors, so there is no effect anymore for --strict/--subjective.
Furthermore, even most WARNs have been bumped to errors, with
WARN only reserved to things that patchew probably ought not
to complain about (and that maintainers probably will notice
anyway during review if they are extreme).

Default to exiting with success even if there are WARN-level
failures, and cause --strict to fail for warnings.  Maintainers
that want to have a strict 80-character limit for their subsystem
can add it to a commit hook for example.

The --subjective synonym is removed.

Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
scripts/checkpatch.pl