lib/PublicInbox/OverIdx.pm | 8 +++----- lib/PublicInbox/SearchIdx.pm | 2 +- diff --git a/lib/PublicInbox/OverIdx.pm b/lib/PublicInbox/OverIdx.pm index 9640f9d13bf57b1e9ae23d48bf9c7c41930b3b4b..f49dfa000431af739eb5e82060c5596fe1af5baf 100644 --- a/lib/PublicInbox/OverIdx.pm +++ b/lib/PublicInbox/OverIdx.pm @@ -216,15 +216,13 @@ $tid; } sub parse_references ($$$) { - my ($smsg, $mid0, $mids) = @_; - my $mime = $smsg->{mime}; - my $hdr = $mime->header_obj; + my ($smsg, $hdr, $mids) = @_; my $refs = references($hdr); push(@$refs, @$mids) if scalar(@$mids) > 1; return $refs if scalar(@$refs) == 0; # prevent circular references here: - my %seen = ( $mid0 => 1 ); + my %seen = ( $smsg->{mid} => 1 ); my @keep; foreach my $ref (@$refs) { if (length($ref) > PublicInbox::MID::MAX_MID_SIZE) { @@ -258,7 +256,7 @@ blob => $oid, }, 'PublicInbox::Smsg'; my $hdr = $mime->header_obj; my $mids = mids_for_index($hdr); - my $refs = parse_references($smsg, $mid0, $mids); + my $refs = parse_references($smsg, $hdr, $mids); my $subj = $smsg->subject; my $xpath; if ($subj ne '') { diff --git a/lib/PublicInbox/SearchIdx.pm b/lib/PublicInbox/SearchIdx.pm index c0578809231800b3fb77e01556ecb1c20bd4b672..32be9c3ffbb63963f64c1958038f9b4c1d6899ba 100644 --- a/lib/PublicInbox/SearchIdx.pm +++ b/lib/PublicInbox/SearchIdx.pm @@ -339,9 +339,9 @@ index_text($self, join(' ', @long), 1, 'XM'); } } $smsg->{to} = $smsg->{cc} = ''; - PublicInbox::OverIdx::parse_references($smsg, $mid0, $mids); $smsg->{blob} = $oid; $smsg->{mid} = $mid0; + PublicInbox::OverIdx::parse_references($smsg, $hdr, $mids); my $data = $smsg->to_doc_data; $doc->set_data($data); if (my $altid = $self->{-altid}) {