Eric SunshineThis character class, like many others in this script, matches horizontal whitespace consisting of spaces and tabs, however, a few extra, entirely harmless, spaces somehow slipped into the expression. Removing them is purely a cosmetic fix.
While at it, re-indent three lines with a single TAB each which were incorrectly indented with six spaces. Also, a purely cosmetic fix.
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com> ---
Just something I noticed while examining t/chainlint.sed after reading Jonathan's report[1] of a false-positive.
This is atop 'es/chain-lint-in-subshell' in 'next'.
[1]: https://public-inbox.org/git/20180730181356.GA156463@aiede.svl.corp.google.com/
t/chainlint.sed | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
Show changes to t/chainlint.sed +4 −4
diff --git a/t/chainlint.sed b/t/chainlint.sed index a0de8a3882..5f0882cb38 100644 --- a/t/chainlint.sed +++ b/t/chainlint.sed @@ -71,9 +71,9 @@ # incomplete line -- slurp up next line :squash /\\$/ { - N - s/\\\n// - bsquash + N + s/\\\n// + bsquash } # here-doc -- swallow it to avoid false hits within its body (but keep the @@ -199,7 +199,7 @@ s/.*\n// # "$(...)" -- command substitution; not closing ")" /\$([^)][^)]*)[^)]*$/bcheckchain # multi-line "$(...\n...)" -- command substitution; treat as nested subshell -/\$([ ]*$/bnest +/\$([ ]*$/bnest # "=(...)" -- Bash array assignment; not closing ")" /=(/bcheckchain # closing "...) &&"
-- 2.18.0.597.ga71716f1ad