user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
* [PATCH] www: add News* wrappers to preload
@ 2016-02-26  3:45 Eric Wong
  0 siblings, 0 replies; only message in thread
From: Eric Wong @ 2016-02-26  3:45 UTC (permalink / raw)
  To: meta

We want to preload as much as possible in -httpd when forking
to save memory via CoW.
---
 lib/PublicInbox/WWW.pm | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/lib/PublicInbox/WWW.pm b/lib/PublicInbox/WWW.pm
index fd09280..2da819b 100644
--- a/lib/PublicInbox/WWW.pm
+++ b/lib/PublicInbox/WWW.pm
@@ -94,12 +94,11 @@ sub preload {
 	require Digest::SHA;
 	require POSIX;
 
-	eval {
-		require PublicInbox::Search;
-		require PublicInbox::SearchView;
-		require PublicInbox::Mbox;
-		require IO::Compress::Gzip;
-	};
+	foreach (qw(PublicInbox::Search PublicInbox::SearchView
+			PublicInbox::Mbox IO::Compress::Gzip
+			PublicInbox::NewsWWW PublicInbox::NewsGroup)) {
+		eval "require $_;";
+	}
 }
 
 # private functions below
-- 
EW


^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2016-02-26  3:45 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-26  3:45 [PATCH] www: add News* wrappers to preload 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).