user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download mbox.gz: |
* [PATCH 7/7] lei_to_mail: call PublicInbox::IPC::DESTROY
    2021-01-20  5:04  4% ` [PATCH 0/7] lei: fixes piled higher and deeper Eric Wong
@ 2021-01-20  5:04  7% ` Eric Wong
  1 sibling, 0 replies; 2+ results
From: Eric Wong @ 2021-01-20  5:04 UTC (permalink / raw)
  To: meta

It doesn't seem to matter at the moment, but it should
save us from some surprises down the line.
---
 lib/PublicInbox/LeiToMail.pm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/PublicInbox/LeiToMail.pm b/lib/PublicInbox/LeiToMail.pm
index 9d9b5748..1be0b09c 100644
--- a/lib/PublicInbox/LeiToMail.pm
+++ b/lib/PublicInbox/LeiToMail.pm
@@ -471,6 +471,7 @@ sub DESTROY {
 	for my $pid_git (grep(/\A$$\0/, keys %$self)) {
 		$self->{$pid_git}->async_wait_all;
 	}
+	$self->SUPER::DESTROY; # PublicInbox::IPC
 }
 
 1;

^ permalink raw reply related	[relevance 7%]

* [PATCH 0/7] lei: fixes piled higher and deeper
  @ 2021-01-20  5:04  4% ` Eric Wong
  2021-01-20  5:04  7% ` [PATCH 7/7] lei_to_mail: call PublicInbox::IPC::DESTROY Eric Wong
  1 sibling, 0 replies; 2+ results
From: Eric Wong @ 2021-01-20  5:04 UTC (permalink / raw)
  To: meta

1/7 was necessary on my FreeBSD 11.x VM
2/7 fixes TEST_RUN_MODE=0
3/7 fixes a long-standing (well, several weeks) annoyance
4/7 depended on 3/7, sorta
5/7 should've been done ages ago
6/7 oops :x
7/7 belts and suspenders

Eric Wong (7):
  lei: allow more mbox inode types
  lei: exit code in oneshot mode
  overidx: eidx_prep: fix leftover dbh reference
  lei q: cleanup store initialization
  lei: dump and clear errors.log in daemon mode
  lei_xsearch: keep l2m->{-wq_s1} while preparing query
  lei_to_mail: call PublicInbox::IPC::DESTROY

 lib/PublicInbox/LEI.pm         | 32 +++++++++++++++++++++++++++-----
 lib/PublicInbox/LeiOverview.pm |  6 ++----
 lib/PublicInbox/LeiQuery.pm    | 18 ++++++++----------
 lib/PublicInbox/LeiToMail.pm   |  5 +++--
 lib/PublicInbox/LeiXSearch.pm  |  4 ++--
 lib/PublicInbox/OverIdx.pm     |  8 +++-----
 t/lei.t                        |  8 +++++++-
 7 files changed, 52 insertions(+), 29 deletions(-)


^ permalink raw reply	[relevance 4%]

Results 1-2 of 2 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2021-01-19  9:34     [PATCH 0/9] lei bugfixes and error handling Eric Wong
2021-01-20  5:04  4% ` [PATCH 0/7] lei: fixes piled higher and deeper Eric Wong
2021-01-20  5:04  7% ` [PATCH 7/7] lei_to_mail: call PublicInbox::IPC::DESTROY Eric Wong

Code repositories for project(s) associated with this public inbox

	https://80x24.org/public-inbox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).