From 95c5010e9abb02973a942f0d4bd707090fed0e5f Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Sat, 27 Dec 2008 02:43:36 -0500 Subject: [PATCH] --- yaml --- r: 125430 b: refs/heads/master c: 46b8af50ba5c072b74740c5fa8ba08e6eabb22f8 h: refs/heads/master v: v3 --- [refs] | 2 +- trunk/scripts/headers_check.pl | 12 +++++++++++- 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/[refs] b/[refs] index edd4d09a76d6..a7127f5f5578 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: 7826005e5a53645d7aab7c13eda76126eadebf0b +refs/heads/master: 46b8af50ba5c072b74740c5fa8ba08e6eabb22f8 diff --git a/trunk/scripts/headers_check.pl b/trunk/scripts/headers_check.pl index 488a3b1f760f..5bdd9753007a 100644 --- a/trunk/scripts/headers_check.pl +++ b/trunk/scripts/headers_check.pl @@ -14,7 +14,9 @@ # Only include files located in asm* and linux* are checked. # The rest are assumed to be system include files. # -# 2) TODO: check for leaked CONFIG_ symbols +# 2) It is checked that prototypes does not use "extern" +# +# 3) TODO: check for leaked CONFIG_ symbols use strict; @@ -33,6 +35,7 @@ while ($line = ) { $lineno++; check_include(); + check_prototypes(); } close FH; } @@ -54,3 +57,10 @@ sub check_include } } } + +sub check_prototypes +{ + if ($line =~ m/^\s*extern\b/) { + printf STDERR "$filename:$lineno: extern's make no sense in userspace\n"; + } +}