]> git.ipfire.org Git - thirdparty/public-inbox.git/commitdiff
inbox: drop $ref arg for writing destination buffer
authorEric Wong <e@80x24.org>
Sat, 7 Jan 2017 22:56:03 +0000 (22:56 +0000)
committerEric Wong <e@80x24.org>
Sat, 7 Jan 2017 23:38:29 +0000 (23:38 +0000)
We never used this feature, so lets drop it for now
since we can have fine-grained memory release with
reference counting, anyways.

lib/PublicInbox/Inbox.pm

index 5503980fc0ca3d297d6062709b3dfacb44aae800..15db929e5109491cb5806ee84e1dc5566be815a8 100644 (file)
@@ -211,16 +211,16 @@ sub nntp_usable {
        $ret;
 }
 
-sub msg_by_path ($$;$) {
+sub msg_by_path ($$) {
        my ($self, $path, $ref) = @_;
        # TODO: allow other refs:
-       my $str = git($self)->cat_file('HEAD:'.$path, $ref);
+       my $str = git($self)->cat_file('HEAD:'.$path);
        $$str =~ s/\A[\r\n]*From [^\r\n]*\r?\n//s if $str;
        $str;
 }
 
-sub msg_by_smsg ($$;$) {
-       my ($self, $smsg, $ref) = @_;
+sub msg_by_smsg ($$) {
+       my ($self, $smsg) = @_;
 
        return unless defined $smsg; # ghost
 
@@ -228,7 +228,7 @@ sub msg_by_smsg ($$;$) {
        # TODO: remove if we bump SCHEMA_VERSION in Search.pm:
        defined(my $blob = $smsg->blob) or return msg_by_mid($self, $smsg->mid);
 
-       my $str = git($self)->cat_file($blob, $ref);
+       my $str = git($self)->cat_file($blob);
        $$str =~ s/\A[\r\n]*From [^\r\n]*\r?\n//s if $str;
        $str;
 }
@@ -238,9 +238,9 @@ sub path_check {
        git($self)->check('HEAD:'.$path);
 }
 
-sub msg_by_mid ($$;$) {
-       my ($self, $mid, $ref) = @_;
-       msg_by_path($self, mid2path($mid), $ref);
+sub msg_by_mid ($$) {
+       my ($self, $mid) = @_;
+       msg_by_path($self, mid2path($mid));
 }
 
 1;