Re: [Parseperl-discuss] Perl::Critic::Policy::Subroutines::RequireFinalReturn

[Chris D. <ch...@ch...>]

[ccing the PPI list]

On Dec 16, 2008, at 7:10 AM, jef...@gm... wrote:

> There seems to be a parsing problem. P::C gives for the following  
> code:
>
> #!/usr/bin/perl
> use warnings;
> use strict;
>
> sub temp {
> print $i++/1;
> return;
> }
>
> Subroutine does not end with "return" at line 5, column 1. See page  
> 197 of PBP. (Severity: 4)
>
> Regards
>
> Jeff

Yes, that appears to be a PPI parse flaw.

       PPI::Statement
         PPI::Token::Word        'print'
         PPI::Token::Whitespace          ' '
         PPI::Token::Symbol      '$i'
         PPI::Token::Operator    '++'
         PPI::Token::Regexp::Match       '/1;\n   return;\n}\n'

Chris