Change 'F.values' calls to 'eigvals(F)'

This commit incorporates changes suggested by @jishnub to edit all associated 'F.values'/'f.values' calls in the diff to 'eigvals(F)'/'eigvals(f)' instead.

Co-authored-by: Jishnu Bhattacharya <[email protected]>

Author: Luis-Varona

src/eigen.jl

@@ -276,8 +276,8 @@ eigvecs(A::Union{Number, AbstractMatrix}; kws...) =
 eigvecs(F::Union{Eigen, GeneralizedEigen}) = F.vectors
 
 eigvals(F::Union{Eigen, GeneralizedEigen}) = F.values
-eigmin(F::Union{Eigen, GeneralizedEigen}) = minimum(F.values)
-eigmax(F::Union{Eigen, GeneralizedEigen}) = maximum(F.values)
+eigmin(F::Union{Eigen, GeneralizedEigen}) = minimum(eigvals(F))
+eigmax(F::Union{Eigen, GeneralizedEigen}) = maximum(eigvals(F))
 
 """
     eigvals!(A; permute::Bool=true, scale::Bool=true, sortby) -> values

test/eigen.jl

@@ -45,9 +45,9 @@ aimg  = randn(n,n)/2
             @test isposdef(a) == isposdef(f)
             @test eigvals(f) === f.values
 
-            if all(isreal, f.values)
-                @test eigmin(f) == minimum(f.values)
-                @test eigmax(f) == maximum(f.values)
+            if all(isreal, eigvals(f))
+                @test eigmin(f) == minimum(eigvals(f))
+                @test eigmax(f) == maximum(eigvals(f))
             else
                 @test_throws MethodError eigmin(f)
                 @test_throws MethodError eigmax(f)
@@ -91,9 +91,9 @@ aimg  = randn(n,n)/2
             @test eigvecs(asym_sg, ASG2) == f.vectors
             @test eigvals(f) === f.values
 
-            if all(isreal, f.values)
-                @test eigmin(f) == minimum(f.values)
-                @test eigmax(f) == maximum(f.values)
+            if all(isreal, eigvals(f))
+                @test eigmin(f) == minimum(eigvals(f))
+                @test eigmax(f) == maximum(eigvals(f))
             else
                 @test_throws MethodError eigmin(f)
                 @test_throws MethodError eigmax(f)