Merge pull request #26 from fkastner/master

fix type instability
anj1 · Dec 4, 2023 · b5fb346 · b5fb346
2 parents b30b1aa + 3639bd8
commit b5fb346
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/Project.toml b/Project.toml
@@ -1,7 +1,7 @@
 name = "AlgebraicNumbers"
 uuid = "e86d093a-a386-599e-b7c7-df0420c8bcba"
 authors = ["Al Nejati", "Felix Kastner"]
-version = "0.1.8"
+version = "0.1.9"
 
 [deps]
 Nemo = "2edaba10-b0f1-5616-af89-8c11ac63239a"

diff --git a/src/algebraic.jl b/src/algebraic.jl
@@ -78,7 +78,7 @@ function calc_precision(coeff::Vector{T}, apprx::Complex{F}) where {T<:Integer,F
 		biginf = convert(F,Inf)
 		n = length(x)
 		if n<=1
-			return Inf
+			return biginf
 		else
 			pdists = [i < j ? abs(x[i]-x[j]) : biginf for i=1:n,j=1:n]
 			return minimum(pdists)
@@ -131,8 +131,7 @@ function simplify(an::AlgebraicNumber)
 	mindists = [minimum(abs.(an.apprx .- prec_roots(get_coeffs(fctr)))) for fctr in fctrs]
 	(newprec, i) = findmin(mindists)
 	fctr = collect(fctrs)[i]
-	an = AlgebraicNumber(get_coeffs(fctr),an.apprx,newprec)
-	return an
+	return AlgebraicNumber(get_coeffs(fctr),an.apprx,newprec)
 end
 
 function ==(an1::AlgebraicNumber,an2::AlgebraicNumber)