Skip to content
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

Declaring anon methods in some circumstances results in syntax error #40

Closed
iarna opened this issue Oct 17, 2011 · 3 comments
Closed

Declaring anon methods in some circumstances results in syntax error #40

iarna opened this issue Oct 17, 2011 · 3 comments

Comments

@iarna
Copy link

iarna commented Oct 17, 2011

This appears to have been caused by 4e03081 which moved injected_if_block "from MSM to MS".

This can be reproduced with:

perl -MMethod::Signatures -e '[method () {},1]'

Reverting 4e03081 fixes the error.

@ghost ghost assigned barefootcoder Oct 18, 2011
@barefootcoder
Copy link
Contributor

I was afraid there might be some edge case where it could cause a problem ... let me try reverting it and see if the error on Perl 5.10 comes back.

@barefootcoder
Copy link
Contributor

Well, I can't reproduce the original problem that caused me to make this change in the first place. So I'll just revert it and throw it out as a dev release and see if any of the CPAN Testers report any errors on it.

barefootcoder added a commit that referenced this issue Oct 18, 2011
apparently moving the inject_if_block code from MSM to MS caused an obscure breakage
thanks to larna for reporting it!
I think there's something suspicious about the generated code, but moving the code back fixes it, so I just did that
retested on 5.12 / 5.10 / 5.8 and none of the weirdness (which inpsired the change in the first place) has recurred
@barefootcoder
Copy link
Contributor

Latest dev version should fix it. I will promote to official release by the end of the week, barring problems from CPAN Testers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants