* [PATCH] config: use "publicinboxlimiter" prefix
@ 2016-09-02 21:02 7% Eric Wong
0 siblings, 0 replies; 1+ results
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 [relevance 7%]
Results 1-1 of 1 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2016-09-02 21:02 7% [PATCH] config: use "publicinboxlimiter" prefix 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).