File Coverage

blib/lib/Perl/Lint/Policy/ControlStructures/ProhibitUnlessBlocks.pm
Criterion Covered Total %
statement 29 30 96.6
branch 9 12 75.0
condition n/a
subroutine 6 6 100.0
pod 0 1 0.0
total 44 49 89.8


line stmt bran cond sub pod time code
1             package Perl::Lint::Policy::ControlStructures::ProhibitUnlessBlocks;
2 134     134   70633 use strict;
  134         167  
  134         3168  
3 134     134   406 use warnings;
  134         153  
  134         2596  
4 134     134   774 use Perl::Lint::Constants::Type;
  134         140  
  134         60878  
5 134     134   602 use parent "Perl::Lint::Policy";
  134         148  
  134         607  
6              
7             use constant {
8 134         27162 DESC => '"unless" block used',
9             EXPL => [97],
10 134     134   6893 };
  134         163  
11              
12             sub evaluate {
13 5     5 0 8 my ($class, $file, $tokens, $src, $args) = @_;
14              
15 5         5 my @violations;
16 5         19 for (my $i = 0; my $token = $tokens->[$i]; $i++) {
17 102         64 my $token_type = $token->{type};
18              
19 102 100       183 if ($token_type == UNLESS_STATEMENT) {
20 7 100       14 if ($tokens->[++$i]->{type} == LEFT_PAREN) {
21 6         5 my $left_paren_num = 1;
22 6         11 for ($i++; my $token = $tokens->[$i]; $i++) {
23 12         11 my $token_type = $token->{type};
24 12 50       27 if ($token_type == LEFT_PAREN) {
    100          
25 0         0 $left_paren_num++;
26             }
27             elsif ($token_type == RIGHT_PAREN) {
28 6 50       11 last if --$left_paren_num <= 0;
29             }
30             }
31 6 50       12 if ($tokens->[++$i]->{type} == LEFT_BRACE) {
32             push @violations, {
33             filename => $file,
34             line => $token->{line},
35 6         25 description => DESC,
36             explanation => EXPL,
37             policy => __PACKAGE__,
38             };
39             }
40             }
41             }
42             }
43              
44 5         13 return \@violations;
45             }
46              
47             1;
48