From 16d1d110117a2521fcc3304541c937385febd66b Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Mon, 2 May 2016 03:20:22 +0000 Subject: view: disable subject threading Broken threads should be exposed to hopefully encourage people to use proper mail clients which set In-Reply-To headers. --- lib/PublicInbox/Thread.pm | 14 ++++++++++++++ 1 file changed, 14 insertions(+) (limited to 'lib/PublicInbox/Thread.pm') diff --git a/lib/PublicInbox/Thread.pm b/lib/PublicInbox/Thread.pm index 781fffff..44a565ad 100644 --- a/lib/PublicInbox/Thread.pm +++ b/lib/PublicInbox/Thread.pm @@ -12,6 +12,20 @@ package PublicInbox::Thread; use strict; use warnings; use base qw(Mail::Thread); +# WARNING! both these Mail::Thread knobs were found by inspecting +# the Mail::Thread 2.55 source code, and we have some monkey patches +# in PublicInbox::Thread to fix memory leaks. Since Mail::Thread +# appears unmaintained, I suppose it's safe to depend on these +# variables for now: +{ + no warnings 'once'; + # we want strict threads to expose (and hopefully discourage) + # use of broken email clients + $Mail::Thread::nosubject = 1; + # Keep ghosts with only a single direct child, + # don't hide that there may be missing messages. + $Mail::Thread::noprune = 1; +} if ($Mail::Thread::VERSION <= 2.55) { eval q(sub _container_class { 'PublicInbox::Thread::Container' }); -- cgit v1.2.3-24-ge0c7