t/porting/authors.t only needs to pass the Author to checkAUTHORS.pl
authorNicholas Clark <nick@ccl4.org>
Wed, 15 Feb 2012 15:11:42 +0000 (16:11 +0100)
committerNicholas Clark <nick@ccl4.org>
Sat, 18 Feb 2012 22:19:44 +0000 (23:19 +0100)
Commit 3ea0c581844689ab didn't go far enough in pruning the input.
When Porting/checkAUTHORS.pl is invoked with --tap it uses
parse_commits_from_stdin_authors() instead of parse_commits_from_stdin(),
which only looks for lines matching /^Author:/

This reduces runtime by a further 8%.

t/porting/authors.t

index 2e78f92..6685468 100644 (file)
@@ -11,6 +11,6 @@ require 't/test.pl';
 find_git_or_skip('all');
 
 # This is the subset of "pretty=fuller" that checkAUTHORS.pl actually needs:
-system("git log --pretty=format:'commit %H%nAuthor: %an <%ae>%nAuthor Date:%nCommit: %cn <%ce>%n' | $^X Porting/checkAUTHORS.pl --tap -");
+system("git log --pretty=format:'Author: %an <%ae>%n' | $^X Porting/checkAUTHORS.pl --tap -");
 
 # EOF