Skip to content

Commit e093ac1

Browse files
committed
Remove unnecessary secondary results
1 parent ada4b2f commit e093ac1

2 files changed

Lines changed: 4 additions & 10 deletions

File tree

src/flint/types/fmpq_mpoly.pyx

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -605,7 +605,6 @@ cdef class fmpq_mpoly(flint_mpoly):
605605
"""
606606
cdef:
607607
fmpq_mpoly res
608-
fmpq_mpoly res2
609608
slong i, nargs
610609

611610
partial_args = tuple((i, dict_args[x]) for i, x in enumerate(self.ctx.names()) if x in dict_args)
@@ -625,13 +624,11 @@ cdef class fmpq_mpoly(flint_mpoly):
625624

626625
# Partial application with args in Z. We evaluate the polynomial one variable at a time
627626
res = create_fmpq_mpoly(self.ctx)
628-
res2 = create_fmpq_mpoly(self.ctx)
629627

630-
fmpq_mpoly_set(res2.val, self.val, self.ctx.val)
628+
fmpq_mpoly_set(res.val, self.val, self.ctx.val)
631629
for (i, _), arg in zip(partial_args, args_fmpq):
632-
if fmpq_mpoly_evaluate_one_fmpq(res.val, res2.val, i, (<fmpq>arg).val, self.ctx.val) == 0:
630+
if fmpq_mpoly_evaluate_one_fmpq(res.val, res.val, i, (<fmpq>arg).val, self.ctx.val) == 0:
633631
raise ValueError("Unreasonably large polynomial") # pragma: no cover
634-
fmpq_mpoly_set(res2.val, res.val, self.ctx.val)
635632
return res
636633

637634
def compose(self, *args) -> fmpq_mpoly:

src/flint/types/fmpz_mpoly.pyx

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -594,7 +594,6 @@ cdef class fmpz_mpoly(flint_mpoly):
594594
"""
595595
cdef:
596596
fmpz_mpoly res
597-
fmpz_mpoly res2
598597
slong i, nargs
599598

600599
partial_args = tuple((i, dict_args[x]) for i, x in enumerate(self.ctx.names()) if x in dict_args)
@@ -614,13 +613,11 @@ cdef class fmpz_mpoly(flint_mpoly):
614613

615614
# Partial application with args in Z. We evaluate the polynomial one variable at a time
616615
res = create_fmpz_mpoly(self.ctx)
617-
res2 = create_fmpz_mpoly(self.ctx)
618616

619-
fmpz_mpoly_set(res2.val, self.val, self.ctx.val)
617+
fmpz_mpoly_set(res.val, self.val, self.ctx.val)
620618
for (i, _), arg in zip(partial_args, args_fmpz):
621-
if fmpz_mpoly_evaluate_one_fmpz(res.val, res2.val, i, (<fmpz>arg).val, self.ctx.val) == 0:
619+
if fmpz_mpoly_evaluate_one_fmpz(res.val, res.val, i, (<fmpz>arg).val, self.ctx.val) == 0:
622620
raise ValueError("Unreasonably large polynomial") # pragma: no cover
623-
fmpz_mpoly_set(res2.val, res.val, self.ctx.val)
624621
return res
625622

626623
def compose(self, *args) -> fmpz_mpoly:

0 commit comments

Comments
 (0)