kolan
/
CMake
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge topic 'hooks-line-length' 

873f21a pre-commit: Reject C++ code with lines too long
This commit is contained in:
David Cole 2012-07-24 17:02:09 -04:00 committed by CMake Topic Stage
parent 2b3a0dbeff 873f21ad1b
commit 17cb00e5dc
1 changed files with 20 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
Utilities/Git/pre-commit
View File

@ -19,6 +19,26 @@ die() {
exit 1
}
#-------------------------------------------------------------------------------
line_too_long=80
bad=$(regex=".{$line_too_long}" &&
git diff-index --cached HEAD --name-only --diff-filter=AM \
--pickaxe-regex -S"$regex" -- 'Source/*.h' 'Source/*.cxx' |
while read file; do
lines_too_long=$(git diff-index -p --cached HEAD \
--pickaxe-regex -S"$regex" -- "$file")
if echo "$lines_too_long" | egrep -q '^\+'"$regex"; then
echo "$lines_too_long"
fi
done)
test -z "$bad" ||
die 'The following changes add lines too long for our C++ style:
'"$bad"'
Use lines strictly less than '"$line_too_long"' characters in C++ code.'
#-------------------------------------------------------------------------------
if test -z "$HOOKS_ALLOW_KWSYS"; then
# Disallow changes to KWSys
files=$(git diff-index --name-only --cached HEAD -- Source/kwsys) &&