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

Fix compiler handling of type params in stacks (issue #918) #1452

Merged
merged 2 commits into from
Dec 5, 2016
Merged

Conversation

sylvanc
Copy link
Contributor

@sylvanc sylvanc commented Dec 4, 2016

Check a stack of typeparam defs instead of just the original. Fixes #918.

@jemc
Copy link
Member

jemc commented Dec 4, 2016

Can you also add a compiler test for the minimal case presented in #918, so we can prove the fix and prevent regression?

@sylvanc
Copy link
Contributor Author

sylvanc commented Dec 4, 2016

Good call!

@jemc jemc changed the title Fix #918 Fix compiler handling of type params in stacks (issue #918) Dec 5, 2016
@jemc jemc added the changelog - fixed Automatically add "Fixed" CHANGELOG entry on merge label Dec 5, 2016
@jemc jemc merged commit 906dd4b into master Dec 5, 2016
@jemc jemc deleted the fix-918 branch December 5, 2016 01:38
ponylang-main added a commit that referenced this pull request Dec 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog - fixed Automatically add "Fixed" CHANGELOG entry on merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants