diff mbox

[U-Boot] checkpatch: remove unnecessary + after {8,8}

Message ID 1411468862-5886-1-git-send-email-marex@denx.de
State Accepted
Delegated to: Tom Rini
Headers show

Commit Message

Marek Vasut Sept. 23, 2014, 10:41 a.m. UTC 
From: Joe Perches <joe@perches.com>

Pick the following commit from Linux kernel:
commit 66cb4ee0e52ca721f609fd5eec16187189ae5fda
Author: Joe Perches <joe@perches.com>
Date:   Wed Sep 10 09:40:47 2014 +1000

checkpatch: remove unnecessary + after {8,8}

There's a useless "+" use that needs to be removed as perl 5.20 emits a
"Useless use of greediness modifier '+'" message each time it's hit.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Tom Rini <trini@ti.com>
---
 scripts/checkpatch.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tom Rini Sept. 25, 2014, 2:46 p.m. UTC | #1 
On Tue, Sep 23, 2014 at 12:41:02PM +0200, Marek Vasut wrote:

> From: Joe Perches <joe@perches.com>
> 
> Pick the following commit from Linux kernel:
> commit 66cb4ee0e52ca721f609fd5eec16187189ae5fda
> Author: Joe Perches <joe@perches.com>
> Date:   Wed Sep 10 09:40:47 2014 +1000
> 
> checkpatch: remove unnecessary + after {8,8}
> 
> There's a useless "+" use that needs to be removed as perl 5.20 emits a
> "Useless use of greediness modifier '+'" message each time it's hit.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Tom Rini <trini@ti.com>

Applied to u-boot/master, thanks!
diff mbox

Patch

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 3fed5e4..74db2e2 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -2148,7 +2148,7 @@  sub process {
 				"please, no space before tabs\n" . $herevet) &&
 			    $fix) {
 				while ($fixed[$linenr - 1] =~
-					   s/(^\+.*) {8,8}+\t/$1\t\t/) {}
+					   s/(^\+.*) {8,8}\t/$1\t\t/) {}
 				while ($fixed[$linenr - 1] =~
 					   s/(^\+.*) +\t/$1\t/) {}
 			}