[PATCH 1/5] www: minor cleanups to shorten code

[Eric Wong <e@80x24.org>]

Less scrolling is more efficient.
---
 lib/PublicInbox/WWW.pm | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/lib/PublicInbox/WWW.pm b/lib/PublicInbox/WWW.pm
index d1ee2ff..527d213 100644
--- a/lib/PublicInbox/WWW.pm
+++ b/lib/PublicInbox/WWW.pm
@@ -138,7 +138,6 @@ sub mid2blob {
 	my $path = PublicInbox::MID::mid2path($ctx->{mid});
 	my @cmd = ('git', "--git-dir=$ctx->{git_dir}",
 			qw(cat-file blob), "HEAD:$path");
-	my $cmd = join(' ', @cmd);
 	my $pid = open my $fh, '-|';
 	defined $pid or die "fork failed: $!\n";
 	if ($pid == 0) {
@@ -162,8 +161,7 @@ sub get_mid_txt {
 # /$LISTNAME/m/$MESSAGE_ID.html                   -> HTML content (short quotes)
 sub get_mid_html {
 	my ($ctx) = @_;
-	my $x = mid2blob($ctx);
-	return r404() unless $x;
+	my $x = mid2blob($ctx) or return r404();
 
 	require PublicInbox::View;
 	my $pfx = msg_pfx($ctx);
@@ -178,8 +176,8 @@ sub get_mid_html {
 # /$LISTNAME/f/$MESSAGE_ID.html                   -> HTML content (fullquotes)
 sub get_full_html {
 	my ($ctx) = @_;
-	my $x = mid2blob($ctx);
-	return r404() unless $x;
+	my $x = mid2blob($ctx) or return r404();
+
 	require PublicInbox::View;
 	my $foot = footer($ctx);
 	require Email::MIME;
-- 
EW