DL-Mirrors/kernel-tenderloin-3.0
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

checkpatch: remove 'static const char * const' check

Browse Source 
While this check is valid in theory, it generates a plethora
of false positives on ALSA drivers and ends up causing more trouble
than it saves.

Signed-off-by: Gregory Bean <gbean@codeaurora.org>
This commit is contained in:
Gregory Bean
2011-07-22 11:12:16 -07:00
committed by Bryan Huntsman
parent 675b31fc19
commit f1fdaffaa7
1 changed files with 0 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
scripts/checkpatch.pl
View File

@@ -1959,12 +1959,6 @@ sub process {
$herecurr); $herecurr);
} }
# check for static const char * arrays.
if ($line =~ /\bstatic\s+const\s+char\s*\*\s*(\w+)\s*\[\s*\]\s*=\s*/) {
WARN("static const char * array should probably be static const char * const\n" .
$herecurr);
}
# check for static char foo[] = "bar" declarations. # check for static char foo[] = "bar" declarations.
if ($line =~ /\bstatic\s+char\s+(\w+)\s*\[\s*\]\s*=\s*"/) { if ($line =~ /\bstatic\s+char\s+(\w+)\s*\[\s*\]\s*=\s*"/) {
WARN("static char array declaration should probably be static const char\n" . WARN("static char array declaration should probably be static const char\n" .