From e9b2623034841cdaffb8d6f4831f86f11b04357a Mon Sep 17 00:00:00 2001
From: Eric Wong <e@80x24.org>
Date: Thu, 12 Dec 2019 08:03:39 +0000
Subject: mbox: do not try to create filename from empty string

This was causing warnings to pop up in syslogs for messages with
empty Subject headers.
---
 lib/PublicInbox/Mbox.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'lib/PublicInbox/Mbox.pm')

diff --git a/lib/PublicInbox/Mbox.pm b/lib/PublicInbox/Mbox.pm
index 42cedd15..c50a118b 100644
--- a/lib/PublicInbox/Mbox.pm
+++ b/lib/PublicInbox/Mbox.pm
@@ -18,7 +18,7 @@ use Email::MIME::Encode;
 sub subject_fn ($) {
 	my ($hdr) = @_;
 	my $fn = $hdr->header('Subject');
-	return 'no-subject' unless defined($fn);
+	return 'no-subject' if (!defined($fn) || $fn eq '');
 
 	# no need for full Email::MIME, here
 	if ($fn =~ /=\?/) {
-- 
cgit v1.2.3-24-ge0c7