2016-08-28 17:00:44 +00:00
|
|
|
#!/bin/bash
|
|
|
|
#
|
|
|
|
# Script to check include/test code for common pybind11 code style errors.
|
2016-09-06 04:02:29 +00:00
|
|
|
#
|
|
|
|
# This script currently checks for
|
|
|
|
#
|
|
|
|
# 1. use of tabs instead of spaces
|
2016-12-12 23:17:29 +00:00
|
|
|
# 2. MSDOS-style CRLF endings
|
|
|
|
# 3. trailing spaces
|
|
|
|
# 4. missing space between keyword and parenthesis, e.g.: for(, if(, while(
|
2016-11-08 09:58:22 +00:00
|
|
|
# 5. Missing space between right parenthesis and brace, e.g. 'for (...){'
|
2016-12-12 23:17:29 +00:00
|
|
|
# 6. opening brace on its own line. It should always be on the same line as the
|
|
|
|
# if/while/for/do statment.
|
2016-08-28 17:00:44 +00:00
|
|
|
#
|
|
|
|
# Invoke as: tools/check-style.sh
|
|
|
|
#
|
|
|
|
|
2016-08-28 18:53:04 +00:00
|
|
|
errors=0
|
|
|
|
IFS=$'\n'
|
|
|
|
found=
|
2016-08-30 00:59:48 +00:00
|
|
|
# The mt=41 sets a red background for matched tabs:
|
2016-09-04 14:00:49 +00:00
|
|
|
exec 3< <(GREP_COLORS='mt=41' grep $'\t' include/ tests/*.{cpp,py,h} docs/*.rst -rn --color=always)
|
2016-08-29 23:04:12 +00:00
|
|
|
while read -u 3 f; do
|
2016-08-28 18:53:04 +00:00
|
|
|
if [ -z "$found" ]; then
|
2016-08-28 17:00:44 +00:00
|
|
|
echo -e '\e[31m\e[01mError: found tabs instead of spaces in the following files:\e[0m'
|
|
|
|
found=1
|
2016-08-28 18:53:04 +00:00
|
|
|
errors=1
|
2016-08-28 17:00:44 +00:00
|
|
|
fi
|
|
|
|
|
|
|
|
echo " $f"
|
|
|
|
done
|
|
|
|
|
2016-09-06 04:02:29 +00:00
|
|
|
found=
|
2016-12-12 23:17:29 +00:00
|
|
|
# The mt=41 sets a red background for matched MS-DOS CRLF line endings
|
|
|
|
exec 3< <(GREP_COLORS='mt=41' grep -IUlr $'\r' include/ tests/*.{cpp,py,h} docs/*.rst --color=always)
|
2016-09-06 04:02:29 +00:00
|
|
|
while read -u 3 f; do
|
|
|
|
if [ -z "$found" ]; then
|
2016-12-12 23:17:29 +00:00
|
|
|
echo -e '\e[31m\e[01mError: found CRLF characters in the following files:\e[0m'
|
2016-09-06 04:02:29 +00:00
|
|
|
found=1
|
|
|
|
errors=1
|
|
|
|
fi
|
|
|
|
|
|
|
|
echo " $f"
|
|
|
|
done
|
|
|
|
|
2016-11-08 09:53:30 +00:00
|
|
|
found=
|
2016-12-12 23:17:29 +00:00
|
|
|
# The mt=41 sets a red background for matched trailing spaces
|
|
|
|
exec 3< <(GREP_COLORS='mt=41' grep '\s\+$' include/ tests/*.{cpp,py,h} docs/*.rst -rn --color=always)
|
2016-11-08 09:53:30 +00:00
|
|
|
while read -u 3 f; do
|
|
|
|
if [ -z "$found" ]; then
|
2016-12-12 23:17:29 +00:00
|
|
|
echo -e '\e[31m\e[01mError: found trailing spaces in the following files:\e[0m'
|
2016-11-08 09:53:30 +00:00
|
|
|
found=1
|
|
|
|
errors=1
|
|
|
|
fi
|
|
|
|
|
|
|
|
echo " $f"
|
|
|
|
done
|
|
|
|
|
2016-08-28 18:53:04 +00:00
|
|
|
found=
|
2016-12-12 23:17:29 +00:00
|
|
|
exec 3< <(grep '\<\(if\|for\|while\|catch\)(\|){' include/ tests/*.{cpp,py,h} -rn --color=always)
|
2016-08-29 23:04:12 +00:00
|
|
|
while read -u 3 line; do
|
2016-08-28 18:53:04 +00:00
|
|
|
if [ -z "$found" ]; then
|
|
|
|
echo -e '\e[31m\e[01mError: found the following coding style problems:\e[0m'
|
|
|
|
found=1
|
|
|
|
errors=1
|
|
|
|
fi
|
|
|
|
|
|
|
|
echo " $line"
|
|
|
|
done
|
|
|
|
|
2016-12-12 23:17:29 +00:00
|
|
|
found=
|
|
|
|
exec 3< <(GREP_COLORS='mt=41' grep '^\s*{\s*$' include/ docs/*.rst -rn --color=always)
|
|
|
|
while read -u 3 f; do
|
|
|
|
if [ -z "$found" ]; then
|
|
|
|
echo -e '\e[31m\e[01mError: braces should occur on the same line as the if/while/.. statement. Found issues in the following files: \e[0m'
|
|
|
|
found=1
|
|
|
|
errors=1
|
|
|
|
fi
|
|
|
|
|
|
|
|
echo " $f"
|
|
|
|
done
|
|
|
|
|
2016-08-28 18:53:04 +00:00
|
|
|
exit $errors
|