diff options
author | Brad King <brad.king@kitware.com> | 2012-07-17 12:16:12 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2012-07-17 12:16:12 (GMT) |
commit | 873f21ad1bcbf900025b7cbddb50b94b37d198aa (patch) | |
tree | 6fc1ed8df20b00c102b7b8b071723927bcc237d7 /Utilities/Git | |
parent | d17c58c8534ba68e527a79f33fad60c502ebdde1 (diff) | |
download | CMake-873f21ad1bcbf900025b7cbddb50b94b37d198aa.zip CMake-873f21ad1bcbf900025b7cbddb50b94b37d198aa.tar.gz CMake-873f21ad1bcbf900025b7cbddb50b94b37d198aa.tar.bz2 |
pre-commit: Reject C++ code with lines too long
Check changes in Source/*.h and Source/*.cxx for lines longer than
our style limit.
Diffstat (limited to 'Utilities/Git')
-rwxr-xr-x | Utilities/Git/pre-commit | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/Utilities/Git/pre-commit b/Utilities/Git/pre-commit index 110e9ee..d308a81 100755 --- a/Utilities/Git/pre-commit +++ b/Utilities/Git/pre-commit @@ -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) && |