{"action":"created","issue":{"url":"https://api.github.com/repos/JuliaLang/julia/issues/18419","repository_url":"https://api.github.com/repos/JuliaLang/julia","labels_url":"https://api.github.com/repos/JuliaLang/julia/issues/18419/labels{/name}","comments_url":"https://api.github.com/repos/JuliaLang/julia/issues/18419/comments","events_url":"https://api.github.com/repos/JuliaLang/julia/issues/18419/events","html_url":"https://github.com/JuliaLang/julia/issues/18419","id":175965251,"number":18419,"title":"New method causes resorting of old methods","user":{"login":"timholy","id":1525481,"avatar_url":"https://avatars.githubusercontent.com/u/1525481?v=3","gravatar_id":"","url":"https://api.github.com/users/timholy","html_url":"https://github.com/timholy","followers_url":"https://api.github.com/users/timholy/followers","following_url":"https://api.github.com/users/timholy/following{/other_user}","gists_url":"https://api.github.com/users/timholy/gists{/gist_id}","starred_url":"https://api.github.com/users/timholy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/timholy/subscriptions","organizations_url":"https://api.github.com/users/timholy/orgs","repos_url":"https://api.github.com/users/timholy/repos","events_url":"https://api.github.com/users/timholy/events{/privacy}","received_events_url":"https://api.github.com/users/timholy/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-09-09T09:22:26Z","updated_at":"2016-09-09T12:25:35Z","closed_at":"2016-09-09T12:24:20Z","body":"From julia-users, ref https://groups.google.com/d/msg/julia-users/IsRlOLGeEs8/RRwrs3u2AQAJ\r\n\r\nShortened version:\r\n```julia\r\njulia> @which pi*pi\r\n*(x::Irrational, y::Irrational) at irrationals.jl:88\r\n\r\njulia> module Erratic2\r\n           typealias DummyUnion Union{Irrational,AbstractFloat}\r\n           abstract DummyReal <: Real\r\n           import Base.*\r\n           *(x::DummyUnion, y::DummyReal) = \"wow\"\r\n       end\r\nErratic2\r\n\r\njulia> pi*pi\r\n------------------------------------------------------------------------------------------\r\nErrorException                                          Stacktrace (most recent call last)\r\n[#1] — *(::Irrational{:π}, ::Irrational{:π})\r\n       ⌙ at promotion.jl:256\r\n\r\n* not defined for Irrational{:π}\r\n\r\njulia> @which pi*pi\r\n*{T<:Number}(x::T, y::T) at promotion.jl:256\r\n```\r\nThis arises because of the following reordering of \"old\" methods (highlighting from `kdiff3`):\r\n![methmul](https://cloud.githubusercontent.com/assets/1525481/18381927/426104a6-7644-11e6-902f-2f63e0f95785.png)\r\n\r\nNote that the method at `promotion.jl:256` is now above the method at `irrationals.jl:88`."},"comment":{"url":"https://api.github.com/repos/JuliaLang/julia/issues/comments/245898463","html_url":"https://github.com/JuliaLang/julia/issues/18419#issuecomment-245898463","issue_url":"https://api.github.com/repos/JuliaLang/julia/issues/18419","id":245898463,"user":{"login":"timholy","id":1525481,"avatar_url":"https://avatars.githubusercontent.com/u/1525481?v=3","gravatar_id":"","url":"https://api.github.com/users/timholy","html_url":"https://github.com/timholy","followers_url":"https://api.github.com/users/timholy/followers","following_url":"https://api.github.com/users/timholy/following{/other_user}","gists_url":"https://api.github.com/users/timholy/gists{/gist_id}","starred_url":"https://api.github.com/users/timholy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/timholy/subscriptions","organizations_url":"https://api.github.com/users/timholy/orgs","repos_url":"https://api.github.com/users/timholy/repos","events_url":"https://api.github.com/users/timholy/events{/privacy}","received_events_url":"https://api.github.com/users/timholy/received_events","type":"User","site_admin":false},"created_at":"2016-09-09T12:25:35Z","updated_at":"2016-09-09T12:25:35Z","body":"Thanks for reporting, and pointing out the duplication."}}
{"actor":{"display_login":"timholy"}}