[PATCH] feed: drop needless version check

2020-04-22 Thread Eric Wong
We don't need to be checking inbox versions in parts of the WWW code. Checking the presence of $ibx->over is enough, everywhere. --- lib/PublicInbox/Feed.pm | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/lib/PublicInbox/Feed.pm b/lib/PublicInbox/Feed.pm index 0734

Re: mail header indexing additions

2020-04-22 Thread Konstantin Ryabitsev
On Mon, Apr 20, 2020 at 01:53:17AM +, Eric Wong wrote: > I'm probably going to start indexing List-Id: headers by > default, and have `lid:' be the search prefix for inboxes > which combine multiple lists and may have unstable email > addresses. This would be handy indeed! > Anything else tha

Re: [PATCH 1/2] view: strip omission character from current message in thread view

2020-04-22 Thread Kyle Meyer
Eric Wong writes: > PATCH 2/2 looked fine, some minor nits on this one: [...] > Will squash this in before pushing: Thanks for the review and fixups.

Re: [PATCH 1/2] view: strip omission character from current message in thread view

2020-04-22 Thread Eric Wong
Kyle Meyer wrote: Thanks for this series (and for figuring out how some of this code works :) > +++ b/lib/PublicInbox/View.pm > @@ -322,7 +322,7 @@ sub _th_index_lite { > my $s_s = nr_to_s($nr_s, 'sibling', 'siblings'); > my $s_c = nr_to_s($nr_c, 'reply', 'replies'); > $attr =~