Merge pull request #27 from fkastner/master

fix some errors reported by JET.jl

Project.toml

@@ -1,7 +1,7 @@
 name = "AlgebraicNumbers"
 uuid = "e86d093a-a386-599e-b7c7-df0420c8bcba"
 authors = ["Al Nejati", "Felix Kastner"]
-version = "0.1.9"
+version = "0.1.10"
 
 [deps]
 Nemo = "2edaba10-b0f1-5616-af89-8c11ac63239a"

src/algebraic.jl

@@ -137,7 +137,7 @@ end
 function ==(an1::AlgebraicNumber,an2::AlgebraicNumber)
 	cf1 = an1.coeff
 	cf2 = an2.coeff
-	(cf1/cf1[end])==(cf2/cf2[end]) || return false
+	(cf1./cf1[end])==(cf2./cf2[end]) || return false
 	prec1 = calc_precision(an1.coeff, an1.apprx)
 	prec2 = calc_precision(an2.coeff, an2.apprx)
 	return abs(an1.apprx-an2.apprx)<min(prec1,prec2)
@@ -185,7 +185,7 @@ end
 
 # partially simplify a polynomial b
 # eliminating repeated factors
-reduce_repeated_factors(p::Nemo.ZZPolyRingElem) = prod(keys(Nemo.factor(p)))
+reduce_repeated_factors(p::Nemo.ZZPolyRingElem) = prod([p for (p,e) in Nemo.factor(p)]; init=one(p))
 
 # multiplication
 function *(an1::AlgebraicNumber,an2::AlgebraicNumber)

src/promote.jl

@@ -20,6 +20,6 @@ function convert(::Type{Int64},an::AlgebraicNumber)
 	if length(c)==2 && abs(c[2])==1
 		return convert(Int64, -c[1]*c[2])
 	else
-		throw(InexactError())
+		throw(InexactError(:convert, Int64, an))
 	end
 end