Merge pull request #156 from fchapoton/cylint3

various cython-lint suggestions
sagemath · Feb 1, 2024 · 38024b2 · 38024b2
2 parents 124436e + 3f868f4
commit 38024b2
Show file tree

Hide file tree

Showing 7 changed files with 21 additions and 20 deletions.
diff --git a/cypari2/closure.pyx b/cypari2/closure.pyx
@@ -21,15 +21,15 @@ Examples:
 [0, 1, 8, 27, 64, 125, 216, 343, 512, 729]
 """
 
-#*****************************************************************************
+# ****************************************************************************
 #       Copyright (C) 2015 Jeroen Demeyer <jdemeyer@cage.ugent.be>
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
 # the Free Software Foundation, either version 2 of the License, or
 # (at your option) any later version.
-#                  http://www.gnu.org/licenses/
-#*****************************************************************************
+#                  https://www.gnu.org/licenses/
+# ****************************************************************************
 
 from __future__ import absolute_import, division, print_function
 
@@ -99,7 +99,8 @@ cdef extern from *:
     GEN call_python_func(GEN* args, PyObject* py_func)
 
 
-cdef GEN call_python(GEN arg1, GEN arg2, GEN arg3, GEN arg4, GEN arg5, ulong nargs, ulong py_func):
+cdef GEN call_python(GEN arg1, GEN arg2, GEN arg3, GEN arg4, GEN arg5,
+                     ulong nargs, ulong py_func):
     """
     This function, which will be installed in PARI, is a front-end for
     ``call_python_func_impl``.

diff --git a/cypari2/gen.pyx b/cypari2/gen.pyx
@@ -40,7 +40,7 @@ AUTHORS:
   conversion
 """
 
-#*****************************************************************************
+# ****************************************************************************
 #       Copyright (C) 2006,2010 William Stein <wstein@gmail.com>
 #       Copyright (C) ???? Justin Walker
 #       Copyright (C) ???? Gonzalo Tornaria
@@ -52,8 +52,8 @@ AUTHORS:
 #  Distributed under the terms of the GNU General Public License (GPL)
 #  as published by the Free Software Foundation; either version 2 of
 #  the License, or (at your option) any later version.
-#                  http://www.gnu.org/licenses/
-#*****************************************************************************
+#                  https://www.gnu.org/licenses/
+# ****************************************************************************
 
 from __future__ import absolute_import, division, print_function
 

diff --git a/cypari2/handle_error.pyx b/cypari2/handle_error.pyx
@@ -10,7 +10,7 @@ AUTHORS:
 
 """
 
-#*****************************************************************************
+# ****************************************************************************
 #       Copyright (C) 2013 Peter Bruin
 #       Copyright (C) 2015 Jeroen Demeyer
 #
@@ -19,7 +19,7 @@ AUTHORS:
 # the Free Software Foundation, either version 2 of the License, or
 # (at your option) any later version.
 #                  https://www.gnu.org/licenses/
-#*****************************************************************************
+# ****************************************************************************
 
 from __future__ import absolute_import, division, print_function
 

diff --git a/cypari2/pari_instance.pyx b/cypari2/pari_instance.pyx
@@ -267,13 +267,13 @@ Test that changing the stack size using ``default`` works properly:
 536870912
 """
 
-#*****************************************************************************
+# ****************************************************************************
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
 # the Free Software Foundation, either version 2 of the License, or
 # (at your option) any later version.
-#                  http://www.gnu.org/licenses/
-#*****************************************************************************
+#                  https://www.gnu.org/licenses/
+# ****************************************************************************
 
 from __future__ import absolute_import, division
 
@@ -1318,7 +1318,7 @@ cdef class Pari(Pari_auto):
 
         If the second argument `p` is specified, it must be a prime.
         Then only the local information at `p` is computed and returned.
-        
+
         Examples:
 
         >>> import cypari2

diff --git a/cypari2/paridecl.pxd b/cypari2/paridecl.pxd
@@ -3524,7 +3524,7 @@ cdef extern from *:     # PARI headers already included by types.pxd
     void    shiftaddress_canon(GEN x, long dec)
     long    timer()
     long    timer2()
-    void    traverseheap( void(*f)(GEN, void *), void *data )
+    void    traverseheap(void(*f)(GEN, void *), void *data)
 
     # intnum.c
 

diff --git a/cypari2/stack.pyx b/cypari2/stack.pyx
@@ -3,7 +3,7 @@ Memory management for Gens on the PARI stack or the heap
 ********************************************************
 """
 
-#*****************************************************************************
+# ****************************************************************************
 #       Copyright (C) 2016 Luca De Feo <luca.defeo@polytechnique.edu>
 #       Copyright (C) 2018 Jeroen Demeyer <J.Demeyer@UGent.be>
 #
@@ -12,12 +12,12 @@ Memory management for Gens on the PARI stack or the heap
 # the Free Software Foundation, either version 2 of the License, or
 # (at your option) any later version.
 #                  http://www.gnu.org/licenses/
-#*****************************************************************************
+# ****************************************************************************
 
 from __future__ import absolute_import, division, print_function
 cimport cython
 
-from cpython.ref cimport PyObject, Py_XINCREF, Py_XDECREF
+from cpython.ref cimport PyObject
 from cpython.exc cimport PyErr_SetString
 
 from cysignals.signals cimport (sig_on, sig_off, sig_block, sig_unblock,

diff --git a/cypari2/types.pxd b/cypari2/types.pxd
@@ -11,12 +11,12 @@ linking to PARI. This file should consist only of typedefs and macros
 from PARI's include files.
 """
 
-#*****************************************************************************
+# ****************************************************************************
 #  Distributed under the terms of the GNU General Public License (GPL)
 #  as published by the Free Software Foundation; either version 2 of
 #  the License, or (at your option) any later version.
-#                  http://www.gnu.org/licenses/
-#*****************************************************************************
+#                  https://www.gnu.org/licenses/
+# ****************************************************************************
 
 cdef extern from "pari/pari.h":
     ctypedef unsigned long ulong "pari_ulong"