Skip to content

Commit

Permalink
Fix #11677, invalid redefinition of constant f in tests
Browse files Browse the repository at this point in the history
we clearly need a better long-term strategy for this

[av skip] because there's a separate failure right now, freezing from #10331
  • Loading branch information
tkelman committed Jun 12, 2015
1 parent 190afa6 commit f5f4ea0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/core.jl
Original file line number Diff line number Diff line change
Expand Up @@ -2946,8 +2946,8 @@ immutable T11675{T}
T11675() = new()
end
let x = T11675{Union()}()
function f(x)
function f11675(x)
x.x + 1
end
@test_throws UndefRefError f(x)
@test_throws UndefRefError f11675(x)
end

2 comments on commit f5f4ea0

@JeffBezanson
Copy link
Member

Choose a reason for hiding this comment

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

This shouldn't be necessary; this function is local to the let block but somehow something earlier in the file is making it global.

@JeffBezanson
Copy link
Member

Choose a reason for hiding this comment

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

Ah, I think it is due to #7864.

Please sign in to comment.