diff options
author | Eric Wong <e@80x24.org> | 2023-04-28 21:07:30 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2023-04-29 06:05:13 +0000 |
commit | 847389d8d4012177616e51f0af3151ed8ab58ff3 (patch) | |
tree | db7b1a2fc97af86a658b21bbcf4f4e00d0851686 /lib/PublicInbox/RepoSnapshot.pm | |
parent | 0f3b7b6058cc4b667e6fbd967d882ad8c47de288 (diff) | |
download | public-inbox-847389d8d4012177616e51f0af3151ed8ab58ff3.tar.gz |
This simplifies Git->cat_async_step and fixes Git->async_abort, the latter of which was passing arguments improperly for the --batch-check (or `info') case at the cost of making the few check_async callers handle an extra argument. The extra (PublicInbox::Git) $self argument for check_async callbacks is now gone, as avoiding the temporary cyclic reference doesn't seem worthwhile since the temporary cyclic reference appears in the ->cat_async code paths, too.
Diffstat (limited to 'lib/PublicInbox/RepoSnapshot.pm')
-rw-r--r-- | lib/PublicInbox/RepoSnapshot.pm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/RepoSnapshot.pm b/lib/PublicInbox/RepoSnapshot.pm index 93ba4db6..ebcbbd81 100644 --- a/lib/PublicInbox/RepoSnapshot.pm +++ b/lib/PublicInbox/RepoSnapshot.pm @@ -42,7 +42,7 @@ sub archive_hdr { # parse_hdr for Qspawn } sub ver_check { # git->check_async callback - my ($oid, $type, $size, $ctx) = @_; + my (undef, $oid, $type, $size, $ctx) = @_; return if defined $ctx->{etag}; my $treeish = shift @{$ctx->{-try}} // die 'BUG: no {-try}'; if ($type eq 'missing') { |