Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

Commit

Permalink
Trac #33208: Remove unused code from sage/rings/padics/pow_computer_e…
Browse files Browse the repository at this point in the history
…xt.pyx

The declaration of `tmp` and the reading of its value into the void
below clearly serve no purpose, and raise a cython warning:

{{{#!python
def polynomial(self):
    ...
    cdef ZZ_pX_Modulus_c* tmp
    tmp.val()
    self.restore_top_context()
    cdef ntl_ZZ_pX r = ntl_ZZ_pX.__new__(ntl_ZZ_pX)
    r.c = self.get_top_context()
    r.x = (self.get_top_modulus()[0]).val()
    return r
}}}

URL: https://trac.sagemath.org/33208
Reported by: mjo
Ticket author(s): Michael Orlitzky
Reviewer(s): Frédéric Chapoton
  • Loading branch information
Release Manager committed Feb 5, 2022
2 parents a8f598e + 4f4ae57 commit 7935ebc
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions src/sage/rings/padics/pow_computer_ext.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -793,8 +793,6 @@ cdef class PowComputer_ZZ_pX(PowComputer_ext):
sage: PC.polynomial()
[9765620 0 1]
"""
cdef ZZ_pX_Modulus_c* tmp
tmp.val()
self.restore_top_context()
cdef ntl_ZZ_pX r = ntl_ZZ_pX.__new__(ntl_ZZ_pX)
r.c = self.get_top_context()
Expand Down

0 comments on commit 7935ebc

Please sign in to comment.