diff options
author | Joe Perches <joe@perches.com> | 2015-04-16 15:44:30 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-04-17 09:03:57 -0400 |
commit | 29a3c46673a5e3ee22feaba7020a280486c78f0a (patch) | |
tree | 998900ac47d2f2603c056db112e09551cc5da21f /scripts | |
parent | 2a076f40d8c9be95bee7bcf18436655e1140447f (diff) |
checkpatch: add #define foo "string" long line exception
There are #defines with long string constants like:
#define foo "some really long string > 80 columns"
Add a long line exception for them.
Miscellanea:
Use the $String variable for slightly better readability
Signed-off-by: Joe Perches <joe@perches.com>
Reported-by: Madalin-Cristian Bucur <madalin.bucur@freescale.com>
Cc: Andy Whitcroft <apw@shadowen.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/checkpatch.pl | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 9a8b2bd14dc2..36d7e704af8f 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl | |||
@@ -2513,8 +2513,9 @@ sub process { | |||
2513 | #line length limit | 2513 | #line length limit |
2514 | if ($line =~ /^\+/ && $prevrawline !~ /\/\*\*/ && | 2514 | if ($line =~ /^\+/ && $prevrawline !~ /\/\*\*/ && |
2515 | $rawline !~ /^.\s*\*\s*\@$Ident\s/ && | 2515 | $rawline !~ /^.\s*\*\s*\@$Ident\s/ && |
2516 | !($line =~ /^\+\s*$logFunctions\s*\(\s*(?:(KERN_\S+\s*|[^"]*))?"[X\t]*"\s*(?:|,|\)\s*;)\s*$/ || | 2516 | !($line =~ /^\+\s*$logFunctions\s*\(\s*(?:(KERN_\S+\s*|[^"]*))?$String\s*(?:|,|\)\s*;)\s*$/ || |
2517 | $line =~ /^\+\s*"[^"]*"\s*(?:\s*|,|\)\s*;)\s*$/) && | 2517 | $line =~ /^\+\s*$String\s*(?:\s*|,|\)\s*;)\s*$/ || |
2518 | $line =~ /^\+\s*#\s*define\s+\w+\s+$String$/) && | ||
2518 | $length > $max_line_length) | 2519 | $length > $max_line_length) |
2519 | { | 2520 | { |
2520 | WARN("LONG_LINE", | 2521 | WARN("LONG_LINE", |