fix E261

Author: GiacomoPope

pyproject.toml

@@ -43,7 +43,7 @@ build-backend = "mesonpy"
 # We currently ignore
 # E129 visually indented line with same indent as next logical line
 max-line-length = 120
-ignore = ['E129','E261','E262','E265','E501','E731','E741','E743']
+ignore = ['E129','E262','E265','E501','E731','E741','E743']
 exclude = 'src/flint/flintlib/.*'
 
 [tool.spin]

src/flint/types/acb_poly.pyx

@@ -197,7 +197,7 @@ cdef class acb_poly(flint_poly):
         return u
 
     def __pos__(self):
-        return self # ?
+        return self
 
     def __neg__(s):
         u = acb_poly.__new__(acb_poly)

src/flint/types/arb_poly.pyx

@@ -194,7 +194,7 @@ cdef class arb_poly(flint_poly):
         return u
 
     def __pos__(self):
-        return self # ?
+        return self
 
     def __neg__(s):
         u = arb_poly.__new__(arb_poly)

src/flint/types/fmpq.pyx

@@ -67,7 +67,7 @@ cdef class fmpq(flint_scalar):
 
     def __init__(self, *args):
         if not args:
-            return # zero
+            return  # zero
         elif len(args) == 2:
             p, q = args
         elif len(args) == 1:

src/flint/types/fmpz.pyx

@@ -77,7 +77,7 @@ cdef class fmpz(flint_scalar):
         if typecheck(val, fmpz):
             fmpz_set(self.val, (<fmpz>val).val)
         else:
-            if fmpz_set_any_ref(self.val, val) == FMPZ_UNKNOWN: # XXX
+            if fmpz_set_any_ref(self.val, val) == FMPZ_UNKNOWN:  # XXX
                 if typecheck(val, str):
                     if fmpz_set_str(self.val, chars_from_str(val), 10) != 0:
                         raise ValueError("invalid string for fmpz")

src/flint/types/fmpz_mod_poly.pyx

@@ -39,7 +39,7 @@ cdef class fmpz_mod_poly_ctx:
         # Allow context to be made from fmpz_mod_ctx
         if typecheck(mod, fmpz_mod_ctx):
             self.mod = mod
-        else: # Otherwise attempt to create context from moduli
+        else:  # Otherwise attempt to create context from moduli
             self.mod = fmpz_mod_ctx(mod)
 
     def modulus(self):
@@ -579,7 +579,7 @@ cdef class fmpz_mod_poly(flint_poly):
             fmpz_mod_poly_shift_left(
                 res.val, self.val, n, self.ctx.mod.val
             )
-        else: # do nothing, just copy self
+        else:  # do nothing, just copy self
             fmpz_mod_poly_set(
                 res.val, self.val, self.ctx.mod.val
             )
@@ -612,7 +612,7 @@ cdef class fmpz_mod_poly(flint_poly):
             fmpz_mod_poly_shift_right(
                 res.val, self.val, n, self.ctx.mod.val
             )
-        else: # do nothing, just copy self
+        else:  # do nothing, just copy self
             fmpz_mod_poly_set(
                 res.val, self.val, self.ctx.mod.val
             )
@@ -1004,9 +1004,9 @@ cdef class fmpz_mod_poly(flint_poly):
         length = fmpz_mod_poly_degree(self.val, self.ctx.mod.val)
         res = self.ctx.new_ctype_poly()
 
-        if n <= 0: # return zero
+        if n <= 0:  # return zero
             return res
-        elif n > length: # do nothing
+        elif n > length:  # do nothing
             fmpz_mod_poly_set(
                 res.val, self.val, self.ctx.mod.val
             )

src/flint/types/fq_default_poly.pyx

@@ -31,7 +31,7 @@ cdef class fq_default_poly_ctx:
         # Allow context to be made from fq_default_ctx
         if len(args) == 1 and typecheck(args[0], fq_default_ctx):
             self.field = args[0]
-        else: # Otherwise attempt to create context from moduli
+        else:  # Otherwise attempt to create context from moduli
             self.field = fq_default_ctx(*args, **kwargs)
 
     cdef set_list_as_fq_default_poly(self, fq_default_poly_t poly, val):
@@ -431,9 +431,9 @@ cdef class fq_default_poly(flint_poly):
         length = fq_default_poly_degree(self.val, self.ctx.field.val)
         res = self.ctx.new_ctype_poly()
 
-        if n <= 0: # return zero
+        if n <= 0:  # return zero
             return res
-        elif n > length: # do nothing
+        elif n > length:  # do nothing
             fq_default_poly_set(
                 res.val, self.val, self.ctx.field.val
             )
@@ -923,7 +923,7 @@ cdef class fq_default_poly(flint_poly):
             fq_default_poly_shift_left(
                 res.val, self.val, n, self.ctx.field.val
             )
-        else: # do nothing, just copy self
+        else:  # do nothing, just copy self
             fq_default_poly_set(
                 res.val, self.val, self.ctx.field.val
             )
@@ -956,7 +956,7 @@ cdef class fq_default_poly(flint_poly):
             fq_default_poly_shift_right(
                 res.val, self.val, n, self.ctx.field.val
             )
-        else: # do nothing, just copy self
+        else:  # do nothing, just copy self
             fq_default_poly_set(
                 res.val, self.val, self.ctx.field.val
             )

src/flint/types/nmod_mat.pyx

@@ -198,7 +198,7 @@ cdef class nmod_mat(flint_mat):
         i, j = index
         if i < 0 or i >= self.nrows() or j < 0 or j >= self.ncols():
             raise IndexError("index %i,%i exceeds matrix dimensions" % (i, j))
-        x = nmod(nmod_mat_entry(self.val, i, j), self.modulus()) # XXX: slow
+        x = nmod(nmod_mat_entry(self.val, i, j), self.modulus())  # XXX: slow
         return x
 
     def __setitem__(self, index, value):

src/flint/types/nmod_poly.pyx

@@ -38,7 +38,7 @@ cdef nmod_poly_set_list(nmod_poly_t poly, list val):
     cdef long i, n
     cdef nmod_t mod
     cdef mp_limb_t v
-    nmod_init(&mod, nmod_poly_modulus(poly)) # XXX
+    nmod_init(&mod, nmod_poly_modulus(poly))  # XXX
     n = PyList_GET_SIZE(val)
     nmod_poly_fit_length(poly, n)
     for i from 0 <= i < n: