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

Fatal: Fix CORE leak trampoline for slurping args #22

Merged
merged 1 commit into from May 12, 2013
Merged

Fatal: Fix CORE leak trampoline for slurping args #22

merged 1 commit into from May 12, 2013

Conversation

nthykier
Copy link
Contributor

When the prototype of a CORE sub has a slurping element, the generated
trampoline would fail to handle "extra" arguments.

Signed-off-by: Niels Thykier niels@thykier.net

When the prototype of a CORE sub has a slurping element, the generated
trampoline would fail to handle "extra" arguments.

Signed-off-by: Niels Thykier <niels@thykier.net>
pjf added a commit that referenced this pull request May 12, 2013
pjf added a commit that referenced this pull request May 12, 2013
Fatal: Fix CORE leak trampoline for slurping args
@pjf pjf merged commit b054a01 into Dual-Life:master May 12, 2013
pjf added a commit that referenced this pull request May 13, 2013
@pjf
Copy link
Collaborator

pjf commented May 13, 2013

And released in 2.19. ;)

pjf added a commit that referenced this pull request May 13, 2013
        * BUGFIX: Loading a file that does not change packages while
          autodie in effect no longer causes weird behaviour when
          slurpy built-ins (like open() and unlink()) are called. GH #22
          Thanks to Niels Thykier.

        * TEST: Tests for leak guard failures for slurpy core functions.
@nthykier nthykier deleted the fix-core-leak-calls branch May 17, 2013 17:57
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

Successfully merging this pull request may close these issues.

None yet

2 participants