diff options
author | Eric Wong <e@80x24.org> | 2019-04-16 01:10:09 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2019-04-16 20:58:23 +0000 |
commit | 6e6f7999361925e4c944f308df4bc32a1842cd69 (patch) | |
tree | 2e190ffc0b3ba4be68d82b28cb86a0d9826426ae /lib/PublicInbox/NewsWWW.pm | |
parent | a94349cda5763507d665809676fccc7b34575a21 (diff) | |
download | public-inbox-6e6f7999361925e4c944f308df4bc32a1842cd69.tar.gz |
'$inbox' is more human-readable, so that is for the more human-readable name in most cases. Making our variable naming more consistent should make the code easier-to-review and harder to screw up.
Diffstat (limited to 'lib/PublicInbox/NewsWWW.pm')
-rw-r--r-- | lib/PublicInbox/NewsWWW.pm | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/PublicInbox/NewsWWW.pm b/lib/PublicInbox/NewsWWW.pm index d7fcb0da..8626cf96 100644 --- a/lib/PublicInbox/NewsWWW.pm +++ b/lib/PublicInbox/NewsWWW.pm @@ -44,11 +44,11 @@ sub call { my (undef, @parts) = split(m!/!, $env->{PATH_INFO}); my ($ng, $article) = @parts; my $pi_config = $self->{pi_config}; - if (my $inbox = $pi_config->lookup_newsgroup($ng)) { - my $url = PublicInbox::Hval::prurl($env, $inbox->{url}); + if (my $ibx = $pi_config->lookup_newsgroup($ng)) { + my $url = PublicInbox::Hval::prurl($env, $ibx->{url}); my $code = 301; if (defined $article && $article =~ /\A\d+\z/) { - my $mid = eval { $inbox->mm->mid_for($article) }; + my $mid = eval { $ibx->mm->mid_for($article) }; if (defined $mid) { # article IDs are not stable across clones, # do not encourage caching/bookmarking them |