Reduce allocations in QQFieldElem construction (#1872)

Nemocas · Sep 30, 2024 · 40d7365 · 40d7365
1 parent 05e38da
commit 40d7365
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/flint/fmpq.jl b/src/flint/fmpq.jl
@@ -20,7 +20,7 @@ end
 
 QQFieldElem(a::Rational{T}) where {T <: Integer} = QQFieldElem(numerator(a), denominator(a))
 
-QQFieldElem(a::Integer) = QQFieldElem(ZZRingElem(a), ZZRingElem(1))
+QQFieldElem(a::Integer) = QQFieldElem(flintify(a))
 
 QQFieldElem(a::Integer, b::Integer) = QQFieldElem(ZZRingElem(a), ZZRingElem(b))
 
@@ -1191,7 +1191,7 @@ end
 #
 ###############################################################################
 
-(a::QQField)() = QQFieldElem(ZZRingElem(0), ZZRingElem(1))
+(a::QQField)() = zero(a)
 
 function (a::QQField)(b::Rational)
   # work around Julia bug, https://github.com/JuliaLang/julia/issues/32569