diff options
| author | Ben Woosley <[email protected]> | 2018-05-21 20:20:24 -0700 |
|---|---|---|
| committer | Ben Woosley <[email protected]> | 2018-06-06 02:11:25 -0700 |
| commit | 9d6c9dbb88593dea995072ba812f115a51ea2b4b (patch) | |
| tree | 156f22bdc7275c0875b1895ed60905ec6bd185d6 | |
| parent | Merge #13288: rpc: Remove the need to include rpc/blockchain.cpp in order to ... (diff) | |
| download | discoin-9d6c9dbb88593dea995072ba812f115a51ea2b4b.tar.xz discoin-9d6c9dbb88593dea995072ba812f115a51ea2b4b.zip | |
lint: Add linter to error on #include <*.cpp>
Files should depend on one another by interface, not by implementation.
This checks for quoted includes as well.
With practicalswift
| -rwxr-xr-x | test/lint/lint-includes.sh | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/test/lint/lint-includes.sh b/test/lint/lint-includes.sh index 7cab0ca4d..2c3456b64 100755 --- a/test/lint/lint-includes.sh +++ b/test/lint/lint-includes.sh @@ -33,6 +33,14 @@ for CPP_FILE in $(filter_suffix cpp); do fi done +INCLUDED_CPP_FILES=$(git grep -E "^#include [<\"][^>\"]+\.cpp[>\"]" -- "*.cpp" "*.h") +if [[ ${INCLUDED_CPP_FILES} != "" ]]; then + echo "The following files #include .cpp files:" + echo "${INCLUDED_CPP_FILES}" + echo + EXIT_CODE=1 +fi + EXPECTED_BOOST_INCLUDES=( boost/algorithm/string.hpp boost/algorithm/string/case_conv.hpp |