viewvcs: avoid one ascii_html call

We can reuse its result for the button text.

diff --git a/lib/PublicInbox/ViewVCS.pm b/lib/PublicInbox/ViewVCS.pm
index 72b79ab79cc9fc89d5c2c38ef3cf580082f63707..86bae7d656fbce3d62779328bcaed4f27200f319 100644 (file)
--- a/lib/PublicInbox/ViewVCS.pm
+++ b/lib/PublicInbox/ViewVCS.pm
@@ -247,9 +247,14 @@ EOM
                        my $rows = ($q =~ tr/\n/\n/) + 1;
                        $q = ascii_html($q);
                        my $ibx_url = ibx_url_for($ctx);
-                       my $alt = $ibx_url ? ' '.ascii_html($ibx_url) : '';
-                       $ibx_url = ascii_html($ibx_url) if defined $ibx_url;
-                       $ibx_url //= $upfx;
+                       my $alt;
+                       if (defined $ibx_url) {
+                               $ibx_url = ascii_html($ibx_url);
+                               $alt = ' '.$ibx_url;
+                       } else {
+                               $ibx_url = $upfx;
+                               $alt = '';
+                       }
                        print $zfh <<EOM;
 <hr><form action="$ibx_url"
 id=related><pre>find related emails, including ancestors/descendants/conflicts