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 1/2] inbox: drop references ASAP for search and msgmap
  2016-05-29  4:16  4% [PATCH 0/2] nntp fixes Eric Wong
@ 2016-05-29  4:16  7% ` Eric Wong
  0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2016-05-29  4:16 UTC (permalink / raw)
  To: meta

We can't leave them lingering in the parent process at
all due to the risk of corruption with multiple processes.
---
 lib/PublicInbox/Inbox.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/PublicInbox/Inbox.pm b/lib/PublicInbox/Inbox.pm
index d050dc8..27218de 100644
--- a/lib/PublicInbox/Inbox.pm
+++ b/lib/PublicInbox/Inbox.pm
@@ -86,7 +86,7 @@ sub base_url {
 sub nntp_usable {
 	my ($self) = @_;
 	my $ret = $self->mm && $self->search;
-	weaken_all();
+	$self->{mm} = $self->{search} = undef;
 	$ret;
 }
 

^ permalink raw reply related	[relevance 7%]

* [PATCH 0/2] nntp fixes
@ 2016-05-29  4:16  4% Eric Wong
  2016-05-29  4:16  7% ` [PATCH 1/2] inbox: drop references ASAP for search and msgmap Eric Wong
  0 siblings, 1 reply; 2+ results
From: Eric Wong @ 2016-05-29  4:16 UTC (permalink / raw)
  To: meta

These are fixes for fallout from the NewsGroup class removal
commit 528230a0411897 ("remove redundant NewsGroup class")

Eric Wong (2):
      inbox: drop references ASAP for search and msgmap
      nntp: fix for missing articles/bodies/heads

 lib/PublicInbox/Inbox.pm | 2 +-
 lib/PublicInbox/NNTP.pm  | 2 +-
 t/nntpd.t                | 8 ++++++++
 3 files changed, 10 insertions(+), 2 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 --
2016-05-29  4:16  4% [PATCH 0/2] nntp fixes Eric Wong
2016-05-29  4:16  7% ` [PATCH 1/2] inbox: drop references ASAP for search and msgmap 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).