-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RT 27364: DESTROY and AUTOLOAD don't parse as subs without 'sub' #31
Comments
moregan
added a commit
to moregan/PPI
that referenced
this issue
Feb 12, 2014
moregan
added a commit
to moregan/PPI
that referenced
this issue
Feb 28, 2014
moregan
added a commit
to moregan/PPI
that referenced
this issue
Feb 28, 2014
moregan
added a commit
to moregan/PPI
that referenced
this issue
Feb 28, 2014
A pernicious aspect of this problem is that after an AUTOLOAD or DESTROY without 'sub', PPI stops creating PPI::Statement::Sub, at least sometimes:
And not just one sub gets missed:
The problem goes away (in this example) when #39 is in effect. |
moregan
added a commit
to moregan/PPI
that referenced
this issue
Mar 4, 2014
moregan
added a commit
to moregan/PPI
that referenced
this issue
Mar 6, 2014
moregan
added a commit
to moregan/PPI
that referenced
this issue
Mar 14, 2014
moregan
added a commit
to moregan/PPI
that referenced
this issue
Mar 14, 2014
moregan
added a commit
to moregan/PPI
that referenced
this issue
Mar 14, 2014
moregan
added a commit
to moregan/PPI
that referenced
this issue
Mar 15, 2014
moregan
added a commit
to moregan/PPI
that referenced
this issue
Aug 23, 2014
wchristian
pushed a commit
that referenced
this issue
Nov 1, 2014
wchristian
pushed a commit
that referenced
this issue
Nov 1, 2014
The full version of the fix for this is now released to CPAN as PPI 1.220. |
moregan
added a commit
to moregan/PPI
that referenced
this issue
Nov 11, 2014
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Jeff Thalhammer points out that Perl allows you to omit 'sub' from DESTROY and AUTOLOAD:
but PPI doesn't recognized them as subs unless 'sub' is included:
https://rt.cpan.org/Public/Bug/Display.html?id=27364
The text was updated successfully, but these errors were encountered: