[openpgp-applet] 52/61: Perl::Critic - correct severe violations

2015-08-18 Thread Clement Hermann
This is an automated email from the git hooks/post-receive script.

nodens-guest pushed a commit to annotated tag OpenPGP_Applet-0.9
in repository openpgp-applet.

commit 1f30a1787fc08a52a8349e91ea3cc1bbe9e33e12
Author: Clément Hermann (nodens) 
Date:   Sat Dec 6 21:57:41 2014 +0100

Perl::Critic - correct severe violations

public_crypto :
Variables shouldn't be declared in conditionnal statement, as it is
confusing. Afterwards, it's checked in scalar context so it should be OK if 
it
is defined but empty.
---
 bin/openpgp-applet | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bin/openpgp-applet b/bin/openpgp-applet
index 7fcd7ac..710c103 100755
--- a/bin/openpgp-applet
+++ b/bin/openpgp-applet
@@ -651,7 +651,7 @@ sub public_crypto {
 my $chosen = choose_keys(\@priv_keys, \@pub_keys);
 my $signer = $chosen->{signer};
 my $recipients_ref = $chosen->{recipients};
-my @recipients = @{$recipients_ref} if defined $recipients_ref;
+my @recipients; @recipients = @{$recipients_ref} if defined 
$recipients_ref;
 my $always_trust = $chosen->{always_trust};
 my $hide_recipients = $chosen->{hide_recipients};
 

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/pkg-perl/packages/openpgp-applet.git

___
Pkg-perl-cvs-commits mailing list
Pkg-perl-cvs-commits@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-perl-cvs-commits

[openpgp-applet] 52/61: Perl::Critic - correct severe violations

2015-08-15 Thread Clément Hermann
This is an automated email from the git hooks/post-receive script.

nodens-guest pushed a commit to annotated tag OpenPGP_Applet-0.9
in repository openpgp-applet.

commit 1f30a1787fc08a52a8349e91ea3cc1bbe9e33e12
Author: Clément Hermann (nodens) 
Date:   Sat Dec 6 21:57:41 2014 +0100

Perl::Critic - correct severe violations

public_crypto :
Variables shouldn't be declared in conditionnal statement, as it is
confusing. Afterwards, it's checked in scalar context so it should be OK if 
it
is defined but empty.
---
 bin/openpgp-applet | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bin/openpgp-applet b/bin/openpgp-applet
index 7fcd7ac..710c103 100755
--- a/bin/openpgp-applet
+++ b/bin/openpgp-applet
@@ -651,7 +651,7 @@ sub public_crypto {
 my $chosen = choose_keys(\@priv_keys, \@pub_keys);
 my $signer = $chosen->{signer};
 my $recipients_ref = $chosen->{recipients};
-my @recipients = @{$recipients_ref} if defined $recipients_ref;
+my @recipients; @recipients = @{$recipients_ref} if defined 
$recipients_ref;
 my $always_trust = $chosen->{always_trust};
 my $hide_recipients = $chosen->{hide_recipients};
 

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/pkg-perl/packages/openpgp-applet.git

___
Pkg-perl-cvs-commits mailing list
Pkg-perl-cvs-commits@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-perl-cvs-commits