From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.2 required=3.0 tests=ALL_TRUSTED,BAYES_00, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF shortcircuit=no autolearn=ham autolearn_force=no version=3.4.6 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id A31271F640 for ; Sun, 29 Jan 2023 10:30:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=80x24.org; s=selector1; t=1674988243; bh=pdp5H6zTXWzcSIV9oBp0TeP5trA18wuJvUWGQAqoHD0=; h=From:To:Subject:Date:In-Reply-To:References:From; b=vc6344zQCoyU7ClzYVm/ZZyLjsTyA2d1QdL1+WaqfjxXArgPhQoOAkO6HkMqje8cp PNs7n8N5s1nGygXgioSw0YD6ztyM6mbIkrWLo43j0q1ADAB1WdHtJbMx3BZqefLW6L VJeFy7m1Sd3RoIpAjEREyAljaLlxVZCvWzAp+DCk= From: Eric Wong To: meta@public-inbox.org Subject: [PATCH 2/2] content_digest_dbg: convert to arrayref and limit to lei Date: Sun, 29 Jan 2023 10:30:42 +0000 Message-Id: <20230129103042.3240848-3-e@80x24.org> In-Reply-To: <20230129103042.3240848-1-e@80x24.org> References: <20230129103042.3240848-1-e@80x24.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: Since it's an extremely small class and not subclassed or anything, we'll make it even smaller as an arrayref. We also don't load this for PublicInbox::WWW or anything that runs in public-facing daemons. --- lib/PublicInbox/ContentDigestDbg.pm | 10 ++++++---- lib/PublicInbox/MailDiff.pm | 5 +---- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/lib/PublicInbox/ContentDigestDbg.pm b/lib/PublicInbox/ContentDigestDbg.pm index 899afbbe..5de0ee8a 100644 --- a/lib/PublicInbox/ContentDigestDbg.pm +++ b/lib/PublicInbox/ContentDigestDbg.pm @@ -1,17 +1,19 @@ # Copyright (C) all contributors # License: AGPL-3.0+ +# only loaded in lei package PublicInbox::ContentDigestDbg; # cf. PublicInbox::ContentDigest use v5.12; use Data::Dumper; use PublicInbox::SHA; +$Data::Dumper::Useqq = $Data::Dumper::Terse = 1; -sub new { bless { dig => PublicInbox::SHA->new(256), fh => $_[1] }, __PACKAGE__ } +sub new { bless [ PublicInbox::SHA->new(256), $_[1] ], __PACKAGE__ } sub add { - $_[0]->{dig}->add($_[1]); - print { $_[0]->{fh} } Dumper([split(/^/sm, $_[1])]) or die "print $!"; + $_[0]->[0]->add($_[1]); + print { $_[0]->[1] } Dumper([split(/^/sm, $_[1])]) or die "print $!"; } -sub hexdigest { $_[0]->{dig}->hexdigest; } +sub hexdigest { $_[0]->[0]->hexdigest } 1; diff --git a/lib/PublicInbox/MailDiff.pm b/lib/PublicInbox/MailDiff.pm index 0ed06f9a..a0ecef9f 100644 --- a/lib/PublicInbox/MailDiff.pm +++ b/lib/PublicInbox/MailDiff.pm @@ -4,8 +4,6 @@ package PublicInbox::MailDiff; use v5.12; use File::Temp 0.19 (); # 0.19 for ->newdir use PublicInbox::ContentHash qw(content_digest); -use PublicInbox::ContentDigestDbg; -use Data::Dumper (); use PublicInbox::MsgIter qw(msg_part_text); use PublicInbox::ViewDiff qw(flush_diff); use PublicInbox::GitAsyncCat; @@ -34,12 +32,11 @@ sub dump_eml ($$$) { $eml->each_part(\&write_part, $self); return if $self->{ctx}; # don't need content_digest noise in WWW UI + require PublicInbox::ContentDigestDbg; # XXX is this even useful? perhaps hide it behind a CLI switch open my $fh, '>', "$dir/content_digest" or die "open: $!"; my $dig = PublicInbox::ContentDigestDbg->new($fh); - local $Data::Dumper::Useqq = 1; - local $Data::Dumper::Terse = 1; content_digest($eml, $dig); print $fh "\n", $dig->hexdigest, "\n" or die "print $!"; close $fh or die "close: $!";