Remove +-(UniformScaling, Number) to avoid ambiguous behavior as poin…

…ted (JuliaLang#17607) out in JuliaLang#17083
mfasi · Sep 5, 2016 · 68b636e · 68b636e
1 parent 766d1de
commit 68b636e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/base/deprecated.jl b/base/deprecated.jl
@@ -790,4 +790,9 @@ function delete!(::EnvHash, k::AbstractString, def)
     haskey(ENV,k) ? delete!(ENV,k) : def
 end
 
+@deprecate (+)(J::UniformScaling, x::Number) J.λ + x
+@deprecate (+)(x::Number, J::UniformScaling) x + J.λ
+@deprecate (-)(J::UniformScaling, x::Number) J.λ - x
+@deprecate (-)(x::Number, J::UniformScaling) x - J.λ
+
 # End deprecations scheduled for 0.6
diff --git a/base/linalg/uniformscaling.jl b/base/linalg/uniformscaling.jl
@@ -38,15 +38,11 @@ ishermitian(J::UniformScaling) = isreal(J.λ)
 (+){T}(B::BitArray{2},J::UniformScaling{T}) = Array(B) + J
 (+)(J::UniformScaling, B::BitArray{2})      = J + Array(B)
 (+)(J::UniformScaling, A::AbstractMatrix)   = A + J
-(+)(J::UniformScaling, x::Number)           = J.λ + x
-(+)(x::Number, J::UniformScaling)           = x + J.λ
 
 (-)(J::UniformScaling)                      = UniformScaling(-J.λ)
 (-)(J1::UniformScaling, J2::UniformScaling) = UniformScaling(J1.λ-J2.λ)
 (-)(B::BitArray{2}, J::UniformScaling)      = Array(B) - J
 (-)(J::UniformScaling, B::BitArray{2})      = J - Array(B)
-(-)(J::UniformScaling, x::Number)           = J.λ - x
-(-)(x::Number, J::UniformScaling)           = x - J.λ
 
 for (t1, t2) in ((:UnitUpperTriangular, :UpperTriangular),
                  (:UnitLowerTriangular, :LowerTriangular))

diff --git a/test/linalg/uniformscaling.jl b/test/linalg/uniformscaling.jl
@@ -26,8 +26,6 @@ srand(123)
 α = randn()
 @test α .* UniformScaling(1.0) == UniformScaling(1.0) .* α
 @test UniformScaling(α)./α == UniformScaling(1.0)
-@test α + UniformScaling(1.0) == UniformScaling(1.0) + α
-@test α - UniformScaling(1.0) == -(UniformScaling(1.0) - α)
 @test copy(UniformScaling(one(Float64))) == UniformScaling(one(Float64))
 @test sprint(show,UniformScaling(one(Float32))) == "UniformScaling{Float32}\n1.0*I"