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
Discourage @_
in signatured subroutine
#19346
Conversation
fd78cb8
to
8236755
Compare
Maybe consider a squash & rebase, before merging this? |
c3a80df
to
8dca553
Compare
This is now rebased, but I don't think we should merge it yet until it's had some more review. |
118e703
to
49d74bb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM I cannot wait for starting using it and fix codebase :-)
Using op_parent() for a better context message
Warn for both `goto &SUB` tailcalls and `&SUB` in perl4 style
…res inside signatured subs
…otice caller's args when caller is a signatured sub
49d74bb
to
2d0a1c6
Compare
Having discussed this with PSC, we've decided to make it an |
No actual runtime behavioural change, but if we're intending to cause some change at some future version it would be useful to get this warning in anyway to help flush out code that would otherwise be affected by such a future change.