浏览代码

Allow wide lines if they have svn id tags in them. (This matters for svk: those tags can be *big*.)

svn:r6900
Nick Mathewson 18 年之前
父节点
当前提交
fc9ed724ed
共有 1 个文件被更改,包括 4 次插入5 次删除
  1. 4 5
      contrib/checkSpace.pl

+ 4 - 5
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;