Add unary plus for Rational, add test (JuliaLang#31772)

Liozou · Apr 20, 2019 · 11fc1ec · 11fc1ec
1 parent 4955696
commit 11fc1ec
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/base/rational.jl b/base/rational.jl
@@ -237,7 +237,9 @@ typemax(::Type{Rational{T}}) where {T<:Integer} = one(T)//zero(T)
 
 isinteger(x::Rational) = x.den == 1
 
++(x::Rational) = (+x.num) // x.den
 -(x::Rational) = (-x.num) // x.den
+
 function -(x::Rational{T}) where T<:BitSigned
  x.num == typemin(T) && throw(OverflowError("rational numerator is typemin(T)"))
  (-x.num) // x.den

diff --git a/test/rational.jl b/test/rational.jl
@@ -370,3 +370,10 @@ end
 
 # issue #31396
 @test round(1//2, RoundNearestTiesUp) === 1//1
+
+@testset "Unary plus on Rational (issue #30749)" begin
+ @test +Rational(true) == 1//1
+ @test +Rational(false) == 0//1
+ @test -Rational(true) == -1//1
+ @test -Rational(false) == 0//1
+end