diff options
author | Beat Bolli <bb@drbeat.li> | 2024-03-15 20:46:18 +0100 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2024-03-16 11:08:57 -0700 |
commit | f70bc702e52796933cafa30dde54fe3c07aa2e81 (patch) | |
tree | fc15e2dd50e9d9c0ec23877227711ddab6db46d7 /contrib | |
parent | babf0b89b383116c9b7aee5a2c3dee3ae1237d65 (diff) | |
download | git-f70bc702e52796933cafa30dde54fe3c07aa2e81.tar.gz |
contrib/coverage-diff: avoid redundant pipelines
Merge multiple sed and "grep | awk" invocations, finally use "sort -u"
instead of "sort | uniq".
Signed-off-by: Beat Bolli <dev+git@drbeat.li>
Acked-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'contrib')
-rwxr-xr-x | contrib/coverage-diff.sh | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/contrib/coverage-diff.sh b/contrib/coverage-diff.sh index 4ec419f900..6ce9603568 100755 --- a/contrib/coverage-diff.sh +++ b/contrib/coverage-diff.sh @@ -74,8 +74,7 @@ do sort >uncovered_lines.txt comm -12 uncovered_lines.txt new_lines.txt | - sed -e 's/$/\)/' | - sed -e 's/^/ /' >uncovered_new_lines.txt + sed -e 's/$/\)/' -e 's/^/ /' >uncovered_new_lines.txt grep -q '[^[:space:]]' <uncovered_new_lines.txt && echo $file >>coverage-data.txt && @@ -91,11 +90,7 @@ cat coverage-data.txt echo "Commits introducing uncovered code:" -commit_list=$(cat coverage-data.txt | - grep -E '^[0-9a-f]{7,} ' | - awk '{print $1;}' | - sort | - uniq) +commit_list=$(awk '/^[0-9a-f]{7,}/ { print $1 }' coverage-data.txt | sort -u) ( for commit in $commit_list |