about summary refs log tree commit homepage
path: root/lib/PublicInbox/AltId.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2019-04-16 01:10:09 +0000
committerEric Wong <e@80x24.org>2019-04-16 20:58:23 +0000
commit6e6f7999361925e4c944f308df4bc32a1842cd69 (patch)
tree2e190ffc0b3ba4be68d82b28cb86a0d9826426ae /lib/PublicInbox/AltId.pm
parenta94349cda5763507d665809676fccc7b34575a21 (diff)
downloadpublic-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/AltId.pm')
-rw-r--r--lib/PublicInbox/AltId.pm8
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/PublicInbox/AltId.pm b/lib/PublicInbox/AltId.pm
index 300bdc0b..c22b9598 100644
--- a/lib/PublicInbox/AltId.pm
+++ b/lib/PublicInbox/AltId.pm
@@ -18,7 +18,7 @@ use URI::Escape qw(uri_unescape);
 # The PREFIX will be a searchable boolean prefix in Xapian
 # Example: serial:gmane:file=/path/to/altmsgmap.sqlite3
 sub new {
-        my ($class, $inbox, $spec, $writable) = @_;
+        my ($class, $ibx, $spec, $writable) = @_;
         my ($type, $prefix, $query) = split(/:/, $spec, 3);
         $type eq 'serial' or die "non-serial not supported, yet\n";
 
@@ -31,10 +31,10 @@ sub new {
         } split(/[&;]/, $query);
         my $f = $params{file} or die "file: required for $type spec $spec\n";
         unless (index($f, '/') == 0) {
-                if (($inbox->{version} || 1) == 1) {
-                        $f = "$inbox->{mainrepo}/public-inbox/$f";
+                if (($ibx->{version} || 1) == 1) {
+                        $f = "$ibx->{mainrepo}/public-inbox/$f";
                 } else {
-                        $f = "$inbox->{mainrepo}/$f";
+                        $f = "$ibx->{mainrepo}/$f";
                 }
         }
         bless {