diff options
author | Guillem Jover <guillem@debian.org> | 2014-09-30 21:25:40 +0200 |
---|---|---|
committer | Guillem Jover <guillem@debian.org> | 2014-10-06 02:08:03 +0200 |
commit | 4502f420bd22eab1634d5a9c710c79d58e13e76e (patch) | |
tree | 7a52377a8f7579b25f97f62ba1e62c8634a9e1b9 /scripts/Dpkg/Control/FieldsCore.pm | |
parent | 386e46306f1f6dbb84296f9be1379de10b1cc535 (diff) | |
download | dpkg-4502f420bd22eab1634d5a9c710c79d58e13e76e.tar.gz |
scripts: Unpack arguments instead of accessing @_ directly
Addresses Subroutines::RequireArgUnpacking.
Warned-by: perlcritic
Diffstat (limited to 'scripts/Dpkg/Control/FieldsCore.pm')
-rw-r--r-- | scripts/Dpkg/Control/FieldsCore.pm | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/scripts/Dpkg/Control/FieldsCore.pm b/scripts/Dpkg/Control/FieldsCore.pm index 3f62b797e..2b7434dc3 100644 --- a/scripts/Dpkg/Control/FieldsCore.pm +++ b/scripts/Dpkg/Control/FieldsCore.pm @@ -428,7 +428,9 @@ Returns true if the field is official and known. =cut sub field_is_official($) { - return exists $FIELDS{field_capitalize($_[0])}; + my $field = field_capitalize(shift); + + return exists $FIELDS{$field}; } =item field_is_allowed_in($fname, @types) @@ -589,7 +591,8 @@ Breaks, ...). Returns undef for fields which are not dependencies. =cut sub field_get_dep_type($) { - my $field = field_capitalize($_[0]); + my $field = field_capitalize(shift); + return unless field_is_official($field); return $FIELDS{$field}{dependency} if exists $FIELDS{$field}{dependency}; return; @@ -603,7 +606,7 @@ FIELD_SEP_SPACE, FIELD_SEP_COMMA or FIELD_SEP_LINE. =cut sub field_get_sep_type($) { - my $field = field_capitalize($_[0]); + my $field = field_capitalize(shift); return $FIELDS{$field}{separator} if exists $FIELDS{$field}{separator}; return FIELD_SEP_UNKNOWN; |