* [PATCH] config: use "publicinboxlimiter" prefix
@ 2016-09-02 21:02 Eric Wong
0 siblings, 0 replies; only message in thread
From: Eric Wong @ 2016-09-02 21:02 UTC (permalink / raw)
To: meta
Just having "limiter" in the prefix may confuse
it with something else. Use the full prefix to
avoid this confusion.
---
lib/PublicInbox/Config.pm | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/lib/PublicInbox/Config.pm b/lib/PublicInbox/Config.pm
index 5adcd0c..8d66cf8 100644
--- a/lib/PublicInbox/Config.pm
+++ b/lib/PublicInbox/Config.pm
@@ -90,8 +90,12 @@ sub limiter {
my ($self, $name) = @_;
$self->{-limiters}->{$name} ||= do {
require PublicInbox::Qspawn;
- my $key = "limiter.$name.max";
- PublicInbox::Qspawn::Limiter->new($self->{$key});
+ my $max;
+ # XXX "limiter.<name>.max" was a historical mistake
+ foreach my $pfx (qw(publicinboxlimiter limiter)) {
+ $max ||= $self->{"$pfx.$name.max"};
+ }
+ PublicInbox::Qspawn::Limiter->new($max);
};
}
--
EW
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2016-09-02 21:02 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-02 21:02 [PATCH] config: use "publicinboxlimiter" prefix Eric Wong
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).