]> git.ipfire.org Git - thirdparty/public-inbox.git/commitdiff
view: reduce ops for <b> encasement
authorEric Wong <e@80x24.org>
Fri, 15 Nov 2024 02:59:31 +0000 (02:59 +0000)
committerEric Wong <e@80x24.org>
Fri, 15 Nov 2024 20:27:44 +0000 (20:27 +0000)
We can rely on print to concatenate its args and
reduce the amount of needless copies and string ops before the
print.

lib/PublicInbox/View.pm

index 7ca85a859b51fdb1e8df1a623ac02f4d764c40ba..ad2592530acd253aa3a5c158b727c2948b0bb5d8 100644 (file)
@@ -272,13 +272,12 @@ sub emit_eml ($$) {
        my $root_anchor = $ctx->{root_anchor} || '';
        my $irt;
        my $obfs_ibx = $ctx->{-obfs_ibx};
-       $subj = '(no subject)' if $subj eq '';
-       $subj = '<b>'.ascii_html($subj).'</b>';
+       $subj = $subj eq '' ? '(no subject)' : ascii_html($subj);
        obfuscate_addrs($obfs_ibx, $subj) if $obfs_ibx;
        $subj = "<u\nid=u>$subj</u>" if $root_anchor eq $id_m;
        my $zfh = $ctx->{zfh} // die 'BUG: no {zfh}';
-       print $zfh "<a\nhref=#e", $id, "\nid=m", $id, '>*</a> ',
-               $subj, "\n", _th_index_lite($mid_raw, \$irt, $id, $ctx);
+       print $zfh "<a\nhref=#e", $id, "\nid=m", $id, '>*</a> <b>',
+               $subj, "</b>\n", _th_index_lite($mid_raw, \$irt, $id, $ctx);
        my @tocc;
        my $ds = delete $smsg->{ds}; # for v1 non-Xapian/SQLite users