Re: [PATCH] update checkpatch.pl to version 0.14

Previous thread: 2.6.24 regression w/ QLA2300 by Alan D. Brunelle on Tuesday, February 5, 2008 - 8:56 am. (7 messages)

Next thread: Re: kara_am by Holger Schurig on Tuesday, February 5, 2008 - 5:53 am. (1 message)
From: Andy Whitcroft
Date: Tuesday, February 5, 2008 - 8:58 am

This version brings the remainder of the queued fixes.  A number of fixes
for items missed reported by Andrew Morton and others.  Also a handful
of new checks and fixes for false positives.  Of note:

 - new warning associated with --file to try and avoid cleanup only patches,
 - corrected handling of completly empty files,
 - corrected report handling with multiple files,
 - handling of possible types in the face of multiple declarations,
 - detection of unnessary braces on complex if statements (where present), and
 - all new comment spacing handling.

Andi Kleen (1):
      Introduce a warning when --file mode is used

Andy Whitcroft (14):
      Version: 0.14
      clean up some space violations in checkpatch.pl
      a completly empty file should not provoke a whinge
      reset report lines buffers between files
      unary ++/-- may abutt close braces
      __typeof__ is also unary
      comments: revamp comment handling
      add --summary-file option adding filename to summary line
      trailing backslashes are not trailing statements
      handle operators passed as parameters such as to ASSERTCMP
      possible types -- enhance debugging
      check for boolean operations with constants
      possible types: handle multiple declarations
      detect and report if statements where all branches are single statements

Arjan van de Ven (1):
      quiet option should not print the summary on no errors

Bartlomiej Zolnierkiewicz (1):
      warn about using __FUNCTION__

Timur Tabi (1):
      loosen spacing checks for __asm__

Signed-off-by: Andy Whitcroft <apw@shadowen.org>
---
 scripts/checkpatch.pl |  226 ++++++++++++++++++++++++++++++++++++++++---------
 1 files changed, 184 insertions(+), 42 deletions(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 545471a..2086a85 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -9,7 +9,7 @@ use strict;
 my $P = $0;
 $P =~ s@.*/@@g;
 
-my $V = '0.13';
+my $V = '0.14';
 
 use ...
From: Benny Halevy
Date: Tuesday, February 5, 2008 - 12:26 pm

Was version 0.13 NACKed?

Benny
--

From: Andy Whitcroft
Date: Wednesday, February 6, 2008 - 7:43 am

It was picked up by Andrew for -mm according to my email.  I do not
think there has been an -mm release since then however.  This patch is
relative to that version.

-apw
--

From: Benny Halevy
Date: Wednesday, February 6, 2008 - 7:53 am

OK. I was afraid it was since Andrew had some comments about white spacing
in checkpatch.pl itself IIRC.

Benny
--

Previous thread: 2.6.24 regression w/ QLA2300 by Alan D. Brunelle on Tuesday, February 5, 2008 - 8:56 am. (7 messages)

Next thread: Re: kara_am by Holger Schurig on Tuesday, February 5, 2008 - 5:53 am. (1 message)