| Filename | /Users/timbo/perl5/perlbrew/perls/perl-5.18.2/lib/site_perl/5.18.2/Perl/Critic/Policy/CodeLayout/RequireTrailingCommas.pm |
| Statements | Executed 18 statements in 398µs |
| Calls | P | F | Exclusive Time |
Inclusive Time |
Subroutine |
|---|---|---|---|---|---|
| 1 | 1 | 1 | 18µs | 18µs | Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@10 |
| 1 | 1 | 1 | 11µs | 345µs | Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@15 |
| 1 | 1 | 1 | 10µs | 19µs | Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@12 |
| 1 | 1 | 1 | 9µs | 21µs | Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@11 |
| 1 | 1 | 1 | 9µs | 31µs | Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@13 |
| 1 | 1 | 1 | 8µs | 64µs | Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@16 |
| 1 | 1 | 1 | 6µs | 7µs | Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::default_severity |
| 1 | 1 | 1 | 2µs | 2µs | Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::supported_parameters |
| 0 | 0 | 0 | 0s | 0s | Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::applies_to |
| 0 | 0 | 0 | 0s | 0s | Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::default_themes |
| 0 | 0 | 0 | 0s | 0s | Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::violates |
| Line | State ments |
Time on line |
Calls | Time in subs |
Code |
|---|---|---|---|---|---|
| 1 | ############################################################################## | ||||
| 2 | # $URL$ | ||||
| 3 | # $Date$ | ||||
| 4 | # $Author$ | ||||
| 5 | # $Revision$ | ||||
| 6 | ############################################################################## | ||||
| 7 | |||||
| 8 | package Perl::Critic::Policy::CodeLayout::RequireTrailingCommas; | ||||
| 9 | |||||
| 10 | 2 | 47µs | 1 | 18µs | # spent 18µs within Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@10 which was called:
# once (18µs+0s) by Module::Pluggable::Object::_require at line 10 # spent 18µs making 1 call to Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@10 |
| 11 | 2 | 23µs | 2 | 34µs | # spent 21µs (9+12) within Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@11 which was called:
# once (9µs+12µs) by Module::Pluggable::Object::_require at line 11 # spent 21µs making 1 call to Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@11
# spent 12µs making 1 call to strict::import |
| 12 | 2 | 21µs | 2 | 28µs | # spent 19µs (10+9) within Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@12 which was called:
# once (10µs+9µs) by Module::Pluggable::Object::_require at line 12 # spent 19µs making 1 call to Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@12
# spent 9µs making 1 call to warnings::import |
| 13 | 2 | 28µs | 2 | 53µs | # spent 31µs (9+22) within Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@13 which was called:
# once (9µs+22µs) by Module::Pluggable::Object::_require at line 13 # spent 31µs making 1 call to Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@13
# spent 22µs making 1 call to Exporter::import |
| 14 | |||||
| 15 | 2 | 25µs | 2 | 678µs | # spent 345µs (11+334) within Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@15 which was called:
# once (11µs+334µs) by Module::Pluggable::Object::_require at line 15 # spent 345µs making 1 call to Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@15
# spent 334µs making 1 call to Exporter::import |
| 16 | 2 | 240µs | 2 | 120µs | # spent 64µs (8+56) within Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@16 which was called:
# once (8µs+56µs) by Module::Pluggable::Object::_require at line 16 # spent 64µs making 1 call to Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::BEGIN@16
# spent 56µs making 1 call to base::import |
| 17 | |||||
| 18 | 1 | 600ns | our $VERSION = '1.121'; | ||
| 19 | |||||
| 20 | #----------------------------------------------------------------------------- | ||||
| 21 | |||||
| 22 | 1 | 2µs | 1 | 30µs | Readonly::Scalar my $DESC => q{List declaration without trailing comma}; # spent 30µs making 1 call to Readonly::Scalar |
| 23 | 1 | 2µs | 1 | 44µs | Readonly::Scalar my $EXPL => [ 17 ]; # spent 44µs making 1 call to Readonly::Scalar |
| 24 | |||||
| 25 | #----------------------------------------------------------------------------- | ||||
| 26 | |||||
| 27 | 1 | 5µs | # spent 2µs within Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::supported_parameters which was called:
# once (2µs+0s) by Perl::Critic::Policy::new at line 88 of Perl/Critic/Policy.pm | ||
| 28 | 1 | 2µs | # spent 7µs (6+1) within Perl::Critic::Policy::CodeLayout::RequireTrailingCommas::default_severity which was called:
# once (6µs+1µs) by Perl::Critic::Policy::get_severity at line 331 of Perl/Critic/Policy.pm | ||
| 29 | sub default_themes { return qw(core pbp cosmetic) } | ||||
| 30 | sub applies_to { return 'PPI::Structure::List' } | ||||
| 31 | |||||
| 32 | #----------------------------------------------------------------------------- | ||||
| 33 | |||||
| 34 | sub violates { | ||||
| 35 | my ( $self, $elem, undef ) = @_; | ||||
| 36 | $elem =~ m{ \n }xms || return; | ||||
| 37 | |||||
| 38 | # Is it an assignment of some kind? | ||||
| 39 | my $sib = $elem->sprevious_sibling(); | ||||
| 40 | return if !$sib; | ||||
| 41 | $sib->isa('PPI::Token::Operator') && $sib =~ m{ = }xms || return; | ||||
| 42 | |||||
| 43 | # List elements are children of an expression | ||||
| 44 | my $expr = $elem->schild(0); | ||||
| 45 | return if !$expr; | ||||
| 46 | |||||
| 47 | # Does the list have more than 1 element? | ||||
| 48 | # This means list element, not PPI element. | ||||
| 49 | my @children = $expr->schildren(); | ||||
| 50 | return if 1 >= grep { $_->isa('PPI::Token::Operator') | ||||
| 51 | && $_ eq $COMMA } @children; | ||||
| 52 | |||||
| 53 | # Is the final element a comma? | ||||
| 54 | my $final = $children[-1]; | ||||
| 55 | if ( ! ($final->isa('PPI::Token::Operator') && $final eq $COMMA) ) { | ||||
| 56 | return $self->violation( $DESC, $EXPL, $elem ); | ||||
| 57 | } | ||||
| 58 | |||||
| 59 | return; #ok! | ||||
| 60 | } | ||||
| 61 | |||||
| 62 | 1 | 3µs | 1; | ||
| 63 | |||||
| 64 | __END__ |