From 261f705649bf5bc9c1954b29cfdb3c894cf44be2 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Thu, 23 Jan 2020 23:05:54 +0000 Subject: contentid: use map to generate %seen for Message-Ids This use of map {} is a common idiom as we no longer consider the Message-ID as part of the digest. --- lib/PublicInbox/ContentId.pm | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/lib/PublicInbox/ContentId.pm b/lib/PublicInbox/ContentId.pm index eb937a0e..0c4a8678 100644 --- a/lib/PublicInbox/ContentId.pm +++ b/lib/PublicInbox/ContentId.pm @@ -60,12 +60,9 @@ sub content_digest ($) { # References: and In-Reply-To: get used interchangeably # in some "duplicates" in LKML. We treat them the same # in SearchIdx, so treat them the same for this: - my %seen; - foreach my $mid (@{mids($hdr)}) { - # do NOT consider the Message-ID as part of the content_id - # if we got here, we've already got Message-ID reuse - $seen{$mid} = 1; - } + # do NOT consider the Message-ID as part of the content_id + # if we got here, we've already got Message-ID reuse + my %seen = map { $_ => 1 } @{mids($hdr)}; foreach my $mid (@{references($hdr)}) { next if $seen{$mid}; $dig->add("ref\0$mid\0"); -- cgit v1.2.3-24-ge0c7