Changed the test for symmetry of a Number. (#18529)

* Changed the test for symmetry of a Number. * added tests for issymmetric with NaN16, NaN32, and NaN as arguments
JuliaLang · Sep 18, 2016 · a648f4a · a648f4a
1 parent 45bc17d
commit a648f4a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/base/linalg/generic.jl b/base/linalg/generic.jl
@@ -712,7 +712,7 @@ function issymmetric(A::AbstractMatrix)
     return true
 end
 
-issymmetric(x::Number) = true
+issymmetric(x::Number) = x == x
 
 """
     ishermitian(A) -> Bool

diff --git a/test/linalg/generic.jl b/test/linalg/generic.jl
@@ -223,6 +223,10 @@ for elty in [Float32,Float64,Complex64,Complex128]
     @test det(a) == a
 end
 
+@test !issymmetric(NaN16)
+@test !issymmetric(NaN32)
+@test !issymmetric(NaN)
+
 @test rank([1.0 0.0; 0.0 0.9],0.95) == 1
 @test qr(big([0 1; 0 0]))[2] == [0 1; 0 0]