lib/PublicInbox/View.pm | 15 +++++---------- diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index d2131245cf74d413d50a3f6d8b16435f915538bb..0331b62c68b0800a3e1af6357d0469e801ceaa41 100644 --- a/lib/PublicInbox/View.pm +++ b/lib/PublicInbox/View.pm @@ -74,7 +74,7 @@ my $mid_raw = $header_obj->header('Message-ID'); my $id = anchor_for($mid_raw); my $seen = $state->{seen}; - $seen->{$id} = "#$id"; # save the anchor for later + $seen->{$id} = "#$id"; # save the anchor for children, later my $mid = PublicInbox::Hval->new_msgid($mid_raw); my $from = PublicInbox::Hval->new_oneline($mime->header('From'))->raw; @@ -88,12 +88,8 @@ my $root_anchor = $state->{root_anchor}; my $path = $root_anchor ? '../../' : ''; my $href = $mid->as_href; my $irt = in_reply_to($header_obj); + my $parent_anchor = $seen->{anchor_for($irt)} if defined $irt; - my ($anchor_idx, $anchor); - if (defined $irt) { - $anchor_idx = anchor_for($irt); - $anchor = $seen->{$anchor_idx}; - } if ($srch) { my $t = $ctx->{flat} ? 'T' : 't'; $subj = "$subj"; @@ -135,13 +131,12 @@ $rv = "\n$more raw "; $rv .= html_footer($mime, 0, undef, $ctx); if (defined $irt) { - unless (defined $anchor) { + unless (defined $parent_anchor) { my $v = PublicInbox::Hval->new_msgid($irt); $v = $v->as_href; - $anchor = "${path}$v/"; - $seen->{$anchor_idx} = $anchor; + $parent_anchor = "${path}$v/"; } - $rv .= " parent"; + $rv .= " parent"; } if ($srch) { if ($ctx->{flat}) {