diff options
author | Nick Mathewson <nickm@torproject.org> | 2006-07-25 22:30:50 +0000 |
---|---|---|
committer | Nick Mathewson <nickm@torproject.org> | 2006-07-25 22:30:50 +0000 |
commit | fc9ed724ed879ace433a248109bf153702daaea8 (patch) | |
tree | 130dd49b1532f2be30edff0a33eb8acd05f3579a /contrib/checkSpace.pl | |
parent | d5a2d51c40bb00f74ff4a3f4082f4a84cfcb8479 (diff) | |
download | tor-fc9ed724ed879ace433a248109bf153702daaea8.tar.gz tor-fc9ed724ed879ace433a248109bf153702daaea8.zip |
Allow wide lines if they have svn id tags in them. (This matters for svk: those tags can be *big*.)
svn:r6900
Diffstat (limited to 'contrib/checkSpace.pl')
-rwxr-xr-x | contrib/checkSpace.pl | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/contrib/checkSpace.pl b/contrib/checkSpace.pl index 30c243e09f..437317f080 100755 --- a/contrib/checkSpace.pl +++ b/contrib/checkSpace.pl @@ -36,11 +36,11 @@ for $fn (@ARGV) { $lastnil = 0; } ## Terminals are still 80 columns wide in my world. I refuse to - ## accept double-line lines. - if (/^.{80}/) { + ## accept double-line lines. Except, of course, svn Id tags + ## can make us go long. + if (/^.{80}/ && !/\$Id: /) { print " Wide:$fn:$.\n"; } - ### Juju to skip over comments and strings, since the tests ### we're about to do are okay there. if ($C) { @@ -88,11 +88,10 @@ for $fn (@ARGV) { } } ## Warn about functions not declared at start of line. - if ($in_func_head || + if ($in_func_head || ($fn !~ /\.h$/ && /^[a-zA-Z0-9_]/ && ! /^(?:static )?(?:typedef|struct|union)[^\(]*$/ && ! /= *\{$/ && ! /;$/)) { - if (/.\{$/){ print "fn() {:$fn:$.\n"; $in_func_head = 0; |