fix #10633, remove redundant definitions for map(Integer, a) etc.

(cherry picked from commit 9075367) ref #18062
JuliaLang · Aug 21, 2016 · 458c832 · 458c832
1 parent 2082241
commit 458c832
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/base/abstractarray.jl b/base/abstractarray.jl
@@ -717,10 +717,6 @@ of_indices(x, y) = similar(dims->y, oftype(indices(x), indices(y)))
 
 full(x::AbstractArray) = x
 
-map(::Type{Integer},  a::Array) = map!(Integer, similar(a,typeof(Integer(one(eltype(a))))), a)
-map(::Type{Signed},   a::Array) = map!(Signed, similar(a,typeof(Signed(one(eltype(a))))), a)
-map(::Type{Unsigned}, a::Array) = map!(Unsigned, similar(a,typeof(Unsigned(one(eltype(a))))), a)
-
 ## range conversions ##
 
 map{T<:Real}(::Type{T}, r::StepRange) = T(r.start):T(r.step):T(last(r))

diff --git a/test/functional.jl b/test/functional.jl
@@ -32,6 +32,10 @@ end
 # maps of strings (character arrays) -- string.jl
 @test map((c)->Char(c+1), "abcDEF") == "bcdEFG"
 
+# issue #10633
+@test isa(map(Integer, Any[1, 2]), Vector{Int})
+@test isa(map(Integer, Any[]), Vector{Integer})
+
 # filter -- array.jl
 @test isequal(filter(x->(x>1), [0 1 2 3 2 1 0]), [2, 3, 2])
 # TODO: @test_throws isequal(filter(x->x+1, [0 1 2 3 2 1 0]), [2, 3, 2])