From 2f1cf63cfbbee843bcc13078bd86f749018a7332 Mon Sep 17 00:00:00 2001 From: Jim Harris Date: Mon, 26 Sep 2022 23:44:47 +0000 Subject: [PATCH] check_format.sh: remove cpp_headers exclusion We don't track any files in test/cpp_headers, so there is no need to do a grep -v to exclude files that contain "cpp_headers". This was a remnant from before we started using git ls-files to determine which files to run through astyle. Fixes: fb87f80c ("scripts/check_format.sh: only check tracked files") Signed-off-by: Jim Harris Change-Id: If08f0bd5c96990dc5e4ca4641c53f70a82590470 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/14689 Tested-by: SPDK CI Jenkins Reviewed-by: Krzysztof Karas Reviewed-by: Aleksey Marchuk Reviewed-by: Konrad Sztyber Reviewed-by: Ben Walker --- scripts/check_format.sh | 2 -- 1 file changed, 2 deletions(-) diff --git a/scripts/check_format.sh b/scripts/check_format.sh index 7a332da81..4d65525d8 100755 --- a/scripts/check_format.sh +++ b/scripts/check_format.sh @@ -127,11 +127,9 @@ function check_c_style() { rm -f astyle.log touch astyle.log git ls-files '*.[ch]' \ - | grep -v cpp_headers \ | xargs -P$(nproc) -n10 astyle --break-return-type --attach-return-type-decl \ --options=.astylerc >> astyle.log git ls-files '*.cpp' '*.cc' '*.cxx' '*.hh' '*.hpp' \ - | grep -v cpp_headers \ | xargs -P$(nproc) -n10 astyle --options=.astylerc >> astyle.log if grep -q "^Formatted" astyle.log; then echo " errors detected"