From 6d8658803b0f75ccdc54d16c8fda4e2cfaed0388 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sat, 11 Jan 2020 22:34:56 +0000 Subject: git: modified: don't slurp `rev-parse --branches' While v1 inboxes typically only have one branch, code repositories may have dozens or even hundreds. Slurping those into memory is a waste. --- lib/PublicInbox/Git.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/PublicInbox') diff --git a/lib/PublicInbox/Git.pm b/lib/PublicInbox/Git.pm index 8ee04e17..0ace907e 100644 --- a/lib/PublicInbox/Git.pm +++ b/lib/PublicInbox/Git.pm @@ -347,7 +347,7 @@ sub modified ($) { my $fh = popen($self, qw(rev-parse --branches)); cat_async_begin($self); local $/ = "\n"; - foreach my $oid (<$fh>) { + while (my $oid = <$fh>) { chomp $oid; cat_async($self, $oid, \&extract_cmt_time, \$modified); } -- cgit v1.2.3-24-ge0c7