Stories
Slash Boxes
Comments
NOTE: use Perl; is on undef hiatus. You can read content, but you can't post it. More info will be forthcoming forthcomingly.

All the Perl that's Practical to Extract and Report

use Perl Log In

Log In

[ Create a new account ]

Alias (5735)

Alias
  (email not shown publicly)
http://ali.as/

Journal of Alias (5735)

Saturday September 02, 2006
02:16 AM

PPI 1.117 released. Perl::Critic hopefully now less broken

[ #30850 ]

I've just uploaded PPI 1.117 to the CPAN.

This release contains a fixes for a number of things I broke in 1.116, but which did not have test cases (and so I didn't notice I had broken).

In particular, trivial compound statements should work again, so that something like...

    LABEL: {
            print "Hello World!";
    } ... is not seen as two statements.

As PPI's biggest "client" (by user count at least) I probably need to get into the habit of installing Perl::Critic on the new release to make sure it still works.

That said, Perl::Critic seems to still be broken with 1.117, but the failures don't look obvious, and so my hope is that the critic guys can release a new version shortly which adapts to the PPI changes/fixes, or that they'll commit more test cases I haven't covered.

Either way, I apologise for the short period of instability to Perl::Critic, hopefully we'll get this stuff cleaned up shortly.

The Fine Print: The following comments are owned by whoever posted them. We are not responsible for them in any way.
 Full
 Abbreviated
 Hidden
More | Login | Reply
Loading... please wait.
  • On behalf of all the Perl::Critic committers I say thanks, Adam. I just committed a fix to Perl::Critic that makes it pass tests with PPI SVN head.

    An anecdote: Perl parses "-t" aggressively. Consider the following code:
        3-t(); # syntax error
        3-test(); # Undefined subroutine &main::test

    PPI doesn't treat the first case the same way Perl does, but it's a syntax error so does it really matter?
    • My standard for that is based on how stable Perl's interpretation is.

      Perl's intepretation changes over time (each version has slight differences here and there). Witness how selfgol only works on a few particular versions of Perl.

      PPI doesn't try to maintain bug for bug compatibility, because it simply can't. It has to be somewhat compatible with all versions.

      So the general direction is that BOTH of $Perl{perl} and $Perl{PPI} trends towards $Perl{Larry}. On the way, $Perl{PPI} also tries to get as close as i
      • Sorry, I was unclear. I didn't mean that "3-t();" anecdote as a criticism of PPI. I just thought I'd mention it here as an oddity that I came across in my testing rather than ignore it. It's just a curiosity.
        • And I didn't take it as a criticism, I was just pointing out I know there's lots of stuff like that, and there are certain conditions under which I fix them, even if illogical.