1. 31 Jul, 2012 4 commits
  2. 01 Jun, 2012 2 commits
  3. 16 Apr, 2012 1 commit
  4. 23 Mar, 2012 12 commits
  5. 07 Feb, 2012 1 commit
    • Joe Perches's avatar
      checkpatch: Warn on code with 6+ tab indentation · 8eef05dd
      Joe Perches authored
      
      
      Overly indented code should be refactored.
      
      Suggest refactoring excessive indentation of of
      if/else/for/do/while/switch statements.
      
      For example:
      
      $ cat t.c
      #include <stdio.h>
      #include <stdlib.h>
      
      int main(int argc, char **argv)
      {
      
      	if (1)
      		if (2)
      			if (3)
      				if (4)
      					if (5)
      						if (6)
      							if (7)
      								if (8)
      									;
      	return 0;
      }
      
      $ ./scripts/checkpatch.pl -f t.c
      WARNING: Too many leading tabs - consider code refactoring
      #12: FILE: t.c:12:
      +						if (6)
      
      WARNING: Too many leading tabs - consider code refactoring
      #13: FILE: t.c:13:
      +							if (7)
      
      WARNING: Too many leading tabs - consider code refactoring
      #14: FILE: t.c:14:
      +								if (8)
      
      total: 0 errors, 3 warnings, 17 lines checked
      
      t.c has style problems, please review.
      
      If any of these errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.
      Signed-off-by: default avatarJoe Perches <joe@perches.com>
      Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
      8eef05dd
  6. 11 Jan, 2012 15 commits
  7. 01 Nov, 2011 2 commits
  8. 15 Sep, 2011 1 commit
  9. 31 Aug, 2011 1 commit
    • Arnaud Lacombe's avatar
      script/checkpatch.pl: warn about deprecated use of EXTRA_{A,C,CPP,LD}FLAGS · c68e5878
      Arnaud Lacombe authored
      Usage of these flags has been deprecated for nearly 4 years by:
      
          commit f77bf014
      
      
          Author: Sam Ravnborg <sam@neptun.(none)>
          Date:   Mon Oct 15 22:25:06 2007 +0200
      
              kbuild: introduce ccflags-y, asflags-y and ldflags-y
      
      Moreover, these flags (at least EXTRA_CFLAGS) have been documented for command
      line use. By default, gmake(1) do not override command line setting, so this is
      likely to result in build failure or unexpected behavior.
      
      Warn about their introduction in Makefile or Kbuild files.
      
      Cc: Sam Ravnborg <sam@ravnborg.org>
      Cc: Andy Whitcroft <apw@canonical.com>
      Signed-off-by: default avatarArnaud Lacombe <lacombar@gmail.com>
      Signed-off-by: default avatarMichal Marek <mmarek@suse.cz>
      c68e5878
  10. 25 Aug, 2011 1 commit