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 #22624, crash in certain circular type definitions #22627

Merged
merged 1 commit into from
Jun 30, 2017

Conversation

JeffBezanson
Copy link
Member

This was caused by expanding a type's field types in the wrong environment.

@JeffBezanson JeffBezanson added backport pending 0.6 bugfix This change fixes an existing bug types and dispatch Types, subtyping and method dispatch labels Jun 30, 2017
test/core.jl Outdated
# issue #22624, more circular definitions
struct T22624{A,B,C}; v::Vector{T22624{Int,A}}; end
let elT = T22624.body.body.body.types[1].parameters[1]
@test elT == T22624{Int64, T22624.var, C} where C
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Int vs Int64 nickel?

This was caused by expanding a type's field types in the wrong
environment.
@JeffBezanson JeffBezanson merged commit 05719b4 into master Jun 30, 2017
@JeffBezanson JeffBezanson deleted the jb/fix22624 branch June 30, 2017 18:32
ararslan pushed a commit that referenced this pull request Sep 11, 2017
fix #22624, crash in certain circular type definitions

Ref #22627
(cherry picked from commit 05719b4)
ararslan pushed a commit that referenced this pull request Sep 13, 2017
This was caused by expanding a type's field types in the wrong
environment.

Ref #22627
(cherry picked from commit 211e8d1)
vtjnash pushed a commit that referenced this pull request Sep 14, 2017
This was caused by expanding a type's field types in the wrong
environment.

Ref #22627
(cherry picked from commit 211e8d1)
ararslan pushed a commit that referenced this pull request Sep 15, 2017
This was caused by expanding a type's field types in the wrong
environment.

Ref #22627
(cherry picked from commit 211e8d1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix This change fixes an existing bug types and dispatch Types, subtyping and method dispatch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants