• Andy Whitcroft's avatar
    update checkpatch.pl to version 0.16 · 773647a0
    Andy Whitcroft authored
    This version brings proper quote tracking across lines, and brings the
    handling of comments into the same mechanism ensuring nesting is correctly
    handled.  It brings the usual flurry of fixes for false positives.  It also
    brings a number of new checks.  The most contentious change will likely be
    the checks for NR_CPUS as this throws some new warnings in kernel/sched.c.
    Of note:
     - all new quote tracking across lines
     - all new comment tracking
     - new more direct, less ambigious wording for some warnings
     - recommends mutexes and completions over semaphores
     - recommends strict_strto* over simple_strto*
     - report on direct use of NR_CPUS
    Andy Whitcroft (22):
          Version: 0.16
          string quote tracking should cross line boundaries
          check spacing round -> correctly across newlines
          checks for linux/ against asm/ include files should be warnings
          standardise on 'required' and 'prohibited'
          take the first end of condition when parsing statements
          values: cope with unbalanced brackets
          preprocessor #elif is not a function
          preprocessor #if should not trigger trailing statement checks
          test: allow us to limit output to a single error
          recommend real mutexes over semaphores
          asm checks should mirror those for __asm__
          warn on semaphores being used in place of completions
          trailing ; on control structure should ignore do {} while ();
          recommend strict_strtoX over simple_strtoX
          redo comment handling as a quote type
          use of NR_CPUS is normally wrong
          consistant spacing should only be about spaces
          if brace check suppression should only apply to the top-levels
          use tr/// to align spacing for operators
          move to using four parameter form of substr
          check and report modifications to include/asm
    Signed-off-by: default avatarAndy Whitcroft <apw@shadowen.org>
    Cc: Ingo Molnar <mingo@elte.hu>
    Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
    Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
checkpatch.pl 53 KB