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

method sorter bug #12092

Closed
vtjnash opened this issue Jul 9, 2015 · 3 comments
Closed

method sorter bug #12092

vtjnash opened this issue Jul 9, 2015 · 3 comments
Assignees
Labels
bug Indicates an unexpected problem or unintended behavior

Comments

@vtjnash
Copy link
Member

vtjnash commented Jul 9, 2015

julia> using Graphics

julia> methods(call, Tuple{Type{Graphics.BoundingBox}, Vararg{Any}})
7-element Array{Any,1}:
 call(::Type{Graphics.BoundingBox}, xmin::Float64, xmax::Float64, ymin::Float64, ymax::Float64) at /home/jameson/.julia/Graphics/src/Graphics.jl:78
--> call(::Type{Graphics.BoundingBox}, points::Graphics.Vec2...) at /home/jameson/.julia/Graphics/src/Graphics.jl:87                                  
--> call(::Type{Graphics.BoundingBox}, bboxes::Graphics.BoundingBox...) at /home/jameson/.julia/Graphics/src/Graphics.jl:98                           
 call(::Type{Graphics.BoundingBox}, xmin, xmax, ymin, ymax) at /home/jameson/.julia/Graphics/src/Graphics.jl:78                                    
--> call(::Type{Graphics.BoundingBox}) at /home/jameson/.julia/Graphics/src/Graphics.jl:84                                                            
 call{T}(::Type{T}, arg) at essentials.jl:56                                                                                                       
 call{T}(::Type{T}, args...) at essentials.jl:57                                                                                                   

julia> methods(call, Tuple{Type{Graphics.BoundingBox}})
1-element Array{Any,1}:
 call(::Type{Graphics.BoundingBox}, points::Graphics.Vec2...) at /home/jameson/.julia/Graphics/src/Graphics.jl:87
@vtjnash vtjnash added the bug Indicates an unexpected problem or unintended behavior label Jul 9, 2015
@JeffBezanson JeffBezanson self-assigned this Jul 10, 2015
@timholy
Copy link
Member

timholy commented Jul 10, 2015

Simpler replicate:

julia> foo(::Type{Float64}, x::Float64) = 1
foo (generic function with 1 method)

julia> foo(::Type{Float64}, x) = 2
foo (generic function with 2 methods)

julia> foo(::Type{Float64}) = 3
foo (generic function with 3 methods)

julia> methods(foo)
# 3 methods for generic function "foo":
foo(::Type{Float64}, x::Float64) at none:1
foo(::Type{Float64}, x) at none:1
foo(::Type{Float64}) at none:1

@timholy
Copy link
Member

timholy commented Jul 10, 2015

Jeff is faster than a refreshing browser.

@vtjnash
Copy link
Member Author

vtjnash commented Jul 10, 2015

very cool. also, this should resolve the last of the type sorter bugs that #8745 was stumbling over! thanks for doing that first big batch.

timholy added a commit that referenced this issue Jan 3, 2016
timholy added a commit that referenced this issue Jan 10, 2016
timholy added a commit that referenced this issue Apr 10, 2016
vtjnash pushed a commit that referenced this issue Apr 29, 2016
vtjnash pushed a commit that referenced this issue May 2, 2016
vtjnash pushed a commit that referenced this issue May 2, 2016
timholy added a commit that referenced this issue May 4, 2016
timholy added a commit that referenced this issue May 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

3 participants