From 50a950ba69c2ba8f2300c9e77f67978979537ad5 Mon Sep 17 00:00:00 2001 From: Bernhard Voelker Date: Sun, 3 Jan 2016 00:18:58 +0100 Subject: maint: fix sc_long_lines to warn about 100+ maximum line length again Since commit v8.23-60-g414a8cf, the above check failed to find sources where the length of the longest line is a 3 or more digit number. * cfg.mk (sc_long_lines): Remove the '\' escape character before '{' and '}', because this would mean literal '{' and '}' characters in an extended regular expression in sed(1). --- cfg.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cfg.mk b/cfg.mk index 441d78911..df22823be 100644 --- a/cfg.mk +++ b/cfg.mk @@ -321,7 +321,7 @@ sc_long_lines: sed -r 1q /dev/null 2>/dev/null \ || { echo "$@: skipping: sed -r not supported"; exit 0; }; \ files=$$($(VC_LIST_EXCEPT) | xargs wc -L | sed -rn '/ total$$/d;\ - s/^ *(8[1-9]|9[0-9]|[0-9]\{3,\}) //p'); \ + s/^ *(8[1-9]|9[0-9]|[0-9]{3,}) //p'); \ halt='line(s) with more than 80 characters; reindent'; \ for file in $$files; do \ expand $$file | grep -nE '^.{80}.' | \ -- cgit v1.2.3-54-g00ecf