2018-09-20 10:01:51 +00:00
|
|
|
#!/usr/bin/env bash
|
|
|
|
|
2020-12-11 19:08:45 +00:00
|
|
|
set -eu
|
|
|
|
|
2021-01-15 19:55:35 +00:00
|
|
|
ERROR_LOG="$(mktemp -t check_style_XXXXXX.log)"
|
2018-11-14 23:05:20 +00:00
|
|
|
|
2020-12-18 23:47:13 +00:00
|
|
|
EXCLUDE_FILES=(
|
2022-11-23 10:51:34 +00:00
|
|
|
# The line below is left unquoted to allow the shell globbing path expansion
|
|
|
|
test/cmdlineTests/*/{err,output}
|
2021-01-15 20:19:58 +00:00
|
|
|
"libsolutil/picosha2.h"
|
2021-03-09 10:34:01 +00:00
|
|
|
"test/cmdlineTests/strict_asm_only_cr/input.yul"
|
2021-01-15 20:19:58 +00:00
|
|
|
"test/libsolutil/UTF8.cpp"
|
|
|
|
"test/libsolidity/syntaxTests/license/license_cr_endings.sol"
|
|
|
|
"test/libsolidity/syntaxTests/license/license_crlf_endings.sol"
|
|
|
|
"test/libsolidity/syntaxTests/license/license_whitespace_trailing.sol"
|
2021-01-19 15:56:27 +00:00
|
|
|
"test/scripts/fixtures/smt_contract_with_crlf_newlines.sol"
|
|
|
|
"test/scripts/fixtures/smt_contract_with_cr_newlines.sol"
|
|
|
|
"test/scripts/fixtures/smt_contract_with_mixed_newlines.sol"
|
2020-12-18 23:47:13 +00:00
|
|
|
)
|
|
|
|
EXCLUDE_FILES_JOINED=$(printf "%s\|" "${EXCLUDE_FILES[@]}")
|
|
|
|
EXCLUDE_FILES_JOINED=${EXCLUDE_FILES_JOINED%??}
|
|
|
|
|
2023-07-11 18:28:21 +00:00
|
|
|
NAMESPACE_STD_FREE_FILES=(
|
|
|
|
libevmasm/*
|
2023-07-12 08:05:47 +00:00
|
|
|
liblangutil/*
|
2023-07-12 12:09:19 +00:00
|
|
|
libsmtutil/*
|
|
|
|
libsolc/*
|
2023-08-14 12:39:16 +00:00
|
|
|
libsolidity/analysis/*
|
|
|
|
libsolidity/ast/*
|
2023-08-14 19:37:31 +00:00
|
|
|
libsolidity/codegen/ir/*
|
|
|
|
libsolidity/codegen/*
|
2023-08-22 13:05:39 +00:00
|
|
|
libsolidity/experimental/*
|
2023-08-14 20:19:11 +00:00
|
|
|
libsolidity/formal/*
|
2023-08-15 08:53:51 +00:00
|
|
|
libsolidity/interface/*
|
2023-08-15 14:21:33 +00:00
|
|
|
libsolidity/lsp/*
|
|
|
|
libsolidity/parsing/*
|
2023-08-17 13:53:20 +00:00
|
|
|
libsolutil/*
|
2023-08-28 08:22:05 +00:00
|
|
|
libyul/*
|
2023-08-23 02:24:09 +00:00
|
|
|
libyul/backends/evm/*
|
2023-08-23 02:03:44 +00:00
|
|
|
libyul/optimiser/*
|
2023-08-23 02:51:26 +00:00
|
|
|
solc/*
|
2023-08-30 00:08:06 +00:00
|
|
|
test/contracts/*
|
|
|
|
test/libevmasm/*
|
|
|
|
test/liblangutil/*
|
|
|
|
test/libsolutil/*
|
|
|
|
test/libsolidity/*
|
|
|
|
test/libsolidity/analysis/*
|
|
|
|
test/libsolidity/interface/*
|
2023-07-11 18:28:21 +00:00
|
|
|
)
|
|
|
|
|
2018-09-20 10:01:51 +00:00
|
|
|
(
|
2018-11-12 14:18:29 +00:00
|
|
|
REPO_ROOT="$(dirname "$0")"/..
|
2021-01-15 20:12:19 +00:00
|
|
|
cd "$REPO_ROOT" || exit 1
|
2018-11-12 14:18:29 +00:00
|
|
|
|
2020-11-18 13:35:16 +00:00
|
|
|
WHITESPACE=$(git grep -n -I -E "^.*[[:space:]]+$" |
|
2023-06-12 10:03:34 +00:00
|
|
|
grep -v "test/libsolidity/ASTJSON\|test/compilationTests/zeppelin/LICENSE\|${EXCLUDE_FILES_JOINED}" || true
|
2020-11-18 13:35:16 +00:00
|
|
|
)
|
2018-09-20 10:01:51 +00:00
|
|
|
|
|
|
|
if [[ "$WHITESPACE" != "" ]]
|
|
|
|
then
|
2021-01-15 20:19:58 +00:00
|
|
|
echo "Error: Trailing whitespace found:" | tee -a "$ERROR_LOG"
|
|
|
|
echo "$WHITESPACE" | tee -a "$ERROR_LOG"
|
2021-01-28 10:42:19 +00:00
|
|
|
scripts/ci/post_style_errors_on_github.sh "$ERROR_LOG"
|
2021-01-15 20:19:58 +00:00
|
|
|
exit 1
|
2018-09-20 10:01:51 +00:00
|
|
|
fi
|
|
|
|
|
2021-10-28 09:58:47 +00:00
|
|
|
function preparedGrep
|
2020-05-07 16:49:54 +00:00
|
|
|
{
|
2021-01-15 20:19:58 +00:00
|
|
|
git grep -nIE "$1" -- '*.h' '*.cpp' | grep -v "${EXCLUDE_FILES_JOINED}"
|
|
|
|
return $?
|
2020-05-07 16:49:54 +00:00
|
|
|
}
|
|
|
|
|
2018-09-20 10:01:51 +00:00
|
|
|
FORMATERROR=$(
|
|
|
|
(
|
2021-01-15 20:19:58 +00:00
|
|
|
preparedGrep "#include \"" | grep -E -v -e "license.h" -e "BuildInfo.h" # Use include with <> characters
|
|
|
|
preparedGrep "\<(if|for|while|switch)\(" # no space after "if", "for", "while" or "switch"
|
|
|
|
preparedGrep "\<for\>\s*\([^=]*\>\s:\s.*\)" # no space before range based for-loop
|
2021-01-15 21:07:08 +00:00
|
|
|
preparedGrep "\<if\>\s*\(.*\)\s*\{\s*$" # "{\n" on same line as "if"
|
2021-05-27 15:38:15 +00:00
|
|
|
preparedGrep "namespace .*\{"
|
2021-01-15 20:19:58 +00:00
|
|
|
preparedGrep "[,\(<]\s*const " # const on left side of type
|
|
|
|
preparedGrep "^\s*(static)?\s*const " # const on left side of type (beginning of line)
|
|
|
|
preparedGrep "^ [^*]|[^*] | [^*]" # uses spaces for indentation or mixes spaces and tabs
|
|
|
|
preparedGrep "[a-zA-Z0-9_]\s*[&][a-zA-Z_]" | grep -E -v "return [&]" # right-aligned reference ampersand (needs to exclude return)
|
|
|
|
# right-aligned reference pointer star (needs to exclude return and comments)
|
|
|
|
preparedGrep "[a-zA-Z0-9_]\s*[*][a-zA-Z_]" | grep -E -v -e "return [*]" -e "^* [*]" -e "^*//.*"
|
2023-03-06 17:20:49 +00:00
|
|
|
# unqualified move()/forward() checks, i.e. make sure that std::move() and std::forward() are used instead of move() and forward()
|
2022-09-01 08:55:29 +00:00
|
|
|
preparedGrep "move\(.+\)" | grep -v "std::move" | grep -E "[^a-z]move"
|
2023-03-06 17:20:49 +00:00
|
|
|
preparedGrep "forward\(.+\)" | grep -v "std::forward" | grep -E "[^a-z]forward"
|
2023-07-11 18:28:21 +00:00
|
|
|
# make sure `using namespace std` is not used in INCLUDE_DIRECTORIES
|
|
|
|
# shellcheck disable=SC2068,SC2068
|
|
|
|
grep -nIE -d skip "using namespace std;" ${NAMESPACE_STD_FREE_FILES[@]}
|
2020-12-11 19:08:45 +00:00
|
|
|
) | grep -E -v -e "^[a-zA-Z\./]*:[0-9]*:\s*\/(\/|\*)" -e "^test/" || true
|
2018-09-20 10:01:51 +00:00
|
|
|
)
|
|
|
|
|
|
|
|
if [[ "$FORMATERROR" != "" ]]
|
|
|
|
then
|
2021-01-15 20:19:58 +00:00
|
|
|
echo "Coding style error:" | tee -a "$ERROR_LOG"
|
|
|
|
echo "$FORMATERROR" | tee -a "$ERROR_LOG"
|
2021-01-28 10:42:19 +00:00
|
|
|
scripts/ci/post_style_errors_on_github.sh "$ERROR_LOG"
|
2021-01-15 20:19:58 +00:00
|
|
|
exit 1
|
2018-09-20 10:01:51 +00:00
|
|
|
fi
|
|
|
|
)
|