summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2006-07-25 22:30:50 +0000
committerNick Mathewson <nickm@torproject.org>2006-07-25 22:30:50 +0000
commitfc9ed724ed879ace433a248109bf153702daaea8 (patch)
tree130dd49b1532f2be30edff0a33eb8acd05f3579a
parentd5a2d51c40bb00f74ff4a3f4082f4a84cfcb8479 (diff)
downloadtor-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
-rwxr-xr-xcontrib/checkSpace.pl9
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;