Просмотр исходного кода

perl: Use croak instead of die on programming errors in modules

Addresses ErrorHandling::RequireCarping.

Warned-by: perlcritic
Guillem Jover лет назад: 13
Родитель
Сommit
4b35d5045a
2 измененных файлов с 4 добавлено и 2 удалено
  1. 2 1
      dselect/methods/Dselect/Ftp.pm
  2. 2 1
      scripts/Dpkg/Control/HashCore.pm

+ 2 - 1
dselect/methods/Dselect/Ftp.pm

@@ -19,6 +19,7 @@ our $VERSION = '0.02';
 
 use Exporter qw(import);
 
+use Carp;
 use Net::FTP;
 use Data::Dumper;
 
@@ -207,7 +208,7 @@ sub yesno($$) {
   $r = -1;
   $r = 0 if $d eq 'n';
   $r = 1 if $d eq 'y';
-  die 'incorrect usage of yesno, stopped' if $r == -1;
+  croak 'incorrect usage of yesno, stopped' if $r == -1;
   while (1) {
     print $msg, " [$d]: ";
     $res = <STDIN>;

+ 2 - 1
scripts/Dpkg/Control/HashCore.pm

@@ -425,6 +425,7 @@ package Dpkg::Control::HashCore::Tie;
 use Dpkg::Checksums;
 use Dpkg::Control::FieldsCore;
 
+use Carp;
 use Tie::Hash;
 use parent -norequire, qw(Tie::ExtraHash);
 
@@ -448,7 +449,7 @@ sub new {
 
 sub TIEHASH  {
     my ($class, $parent) = @_;
-    die 'Parent object must be Dpkg::Control::Hash'
+    croak 'parent object must be Dpkg::Control::Hash'
         if not $parent->isa('Dpkg::Control::HashCore') and
            not $parent->isa('Dpkg::Control::Hash');
     return bless [ {}, $$parent ], $class;