summaryrefslogtreecommitdiff
path: root/tools/ci/jobs/nsiqcppstyle.sh
diff options
context:
space:
mode:
authorFedja Beader <fedja@protonmail.ch>2023-09-10 15:58:24 +0200
committerFedja Beader <fedja@protonmail.ch>2023-09-10 15:58:24 +0200
commit0a2d34be04d72b8982cc2cd488f7e6432d30b40b (patch)
tree8eca42bd37a93b9eae2200133dfd7ad47024ad09 /tools/ci/jobs/nsiqcppstyle.sh
parent3f68449293de30d84e39fef761ca5a4734f99c32 (diff)
downloadplus-0a2d34be04d72b8982cc2cd488f7e6432d30b40b.tar.gz
plus-0a2d34be04d72b8982cc2cd488f7e6432d30b40b.tar.bz2
plus-0a2d34be04d72b8982cc2cd488f7e6432d30b40b.tar.xz
plus-0a2d34be04d72b8982cc2cd488f7e6432d30b40b.zip
trim useless uses of cat
Diffstat (limited to 'tools/ci/jobs/nsiqcppstyle.sh')
-rwxr-xr-xtools/ci/jobs/nsiqcppstyle.sh24
1 files changed, 11 insertions, 13 deletions
diff --git a/tools/ci/jobs/nsiqcppstyle.sh b/tools/ci/jobs/nsiqcppstyle.sh
index a6db88cc9..d4a5dd449 100755
--- a/tools/ci/jobs/nsiqcppstyle.sh
+++ b/tools/ci/jobs/nsiqcppstyle.sh
@@ -14,20 +14,18 @@ rm -rf nsiqcppstyle.git
-e "/debug/" \
> logs/nsiqcppstyle1.log
-cat logs/nsiqcppstyle1.log \
- | grep -v -E \
- -e "gamemodifiers[.]cpp(.+): Incorrect align on long parameter list in front of '(N_|{|})'" \
- -e "graphics_calcImageRect[.]hpp(.+): function [(]calcTileVertexesInline[)] has non named parameter" \
- -e "dye[.]cpp(.+): Do not use goto keyword" \
- -e "item/item[.]cpp(.+): function [(]dragDrop[)] has non named parameter" \
- -e "windowmenu[.]cpp(.+): function [(]addButton[)] has non named parameter" \
- -e "windowmenu[.]cpp(.+): Incorrect align on long parameter list in front of '(.|Visible_true)', it should be aligen in column 15" \
- -e "channeltab[.]h(.+): Incorrect align on long parameter list in front of 'const'" \
- -e "/enums/simpletypes/(.+): function [(]defIntEnum[)] has non named parameter" \
- > "${ERRFILE}"
+grep -v -E \
+ -e "gamemodifiers[.]cpp(.+): Incorrect align on long parameter list in front of '(N_|{|})'" \
+ -e "graphics_calcImageRect[.]hpp(.+): function [(]calcTileVertexesInline[)] has non named parameter" \
+ -e "dye[.]cpp(.+): Do not use goto keyword" \
+ -e "item/item[.]cpp(.+): function [(]dragDrop[)] has non named parameter" \
+ -e "windowmenu[.]cpp(.+): function [(]addButton[)] has non named parameter" \
+ -e "windowmenu[.]cpp(.+): Incorrect align on long parameter list in front of '(.|Visible_true)', it should be aligen in column 15" \
+ -e "channeltab[.]h(.+): Incorrect align on long parameter list in front of 'const'" \
+ -e "/enums/simpletypes/(.+): function [(]defIntEnum[)] has non named parameter" \
+ logs/nsiqcppstyle1.log > "${ERRFILE}"
-export data=$(cat ${ERRFILE})
-if [ "$DATA" != "" ]; then
+if [[ -s "${ERRFILE}" ]]; then
echo "Warnings detected"
cat ${ERRFILE}
fi