Merge pull request #37 from JuliaMath/teh/fix_ambig

Fix ambiguities and test for them

Author: timholy

src/closed.jl

@@ -100,6 +100,14 @@ function convert(::Type{R}, i::ClosedInterval{I}) where {R<:AbstractUnitRange,I<
     R(minimum(i), maximum(i))
 end
 
+# The third is the one we want, but the first two are needed to resolve ambiguities
+Base.Slice{T}(i::ClosedInterval{I}) where {T<:AbstractUnitRange,I<:Integer} =
+    Base.Slice{T}(minimum(i):maximum(i))
+function Base.OneTo{T}(i::ClosedInterval{I}) where {T<:Integer,I<:Integer}
+    @noinline throwstart(i) = throw(ArgumentError("smallest element must be 1, got $(minimum(i))"))
+    minimum(i) == 1 || throwstart(i)
+    Base.OneTo{T}(maximum(i))
+end
 function (::Type{R})(i::ClosedInterval{I}) where {R<:AbstractUnitRange,I<:Integer}
     R(minimum(i), maximum(i))
 end

test/runtests.jl

@@ -3,6 +3,8 @@ using Compat
 using Compat.Test
 using Compat.Dates
 
+@test isempty(detect_ambiguities(IntervalSets, Base, Core))
+
 @testset "IntervalSets" begin
     @test ordered(2, 1) == (1, 2)
     @test ordered(1, 2) == (1, 2)