Fix bug in several Float16 methods where the second positional argume…

…nt wasn't used. Add test based on original issue #17148.
JuliaLang · Jun 27, 2016 · 22a4d8b · 22a4d8b
1 parent 861100c
commit 22a4d8b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/base/float16.jl b/base/float16.jl
@@ -154,7 +154,7 @@ end
 
 for func in (:div,:fld,:cld,:rem,:mod,:atan2,:hypot)
     @eval begin
-        $func(a::Float16,b::Float16) = Float16($func(Float32(a),Float32(a)))
+        $func(a::Float16,b::Float16) = Float16($func(Float32(a),Float32(b)))
     end
 end
 

diff --git a/test/float16.jl b/test/float16.jl
@@ -143,3 +143,6 @@ end
 
 #  #9939 (and #9897)
 @test rationalize(Float16(0.1)) == 1//10
+
+# issue #17148
+@test rem(Float16(1.2), Float16(one(1.2))) == 0.20019531f0