Skip to content

Commit a89c57b

Browse files
author
Release Manager
committed
gh-39092: Remove erroneous member declaration in farey_symbol As you can see in the C++ header file, there is in fact no members named `a, b, c, d`. In fact it's an error in C++ to have a member function and member variable with the same name. Newer versions of Cython gives hard-to-debug errors for this. ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - #12345: short description why this is a dependency --> <!-- - #34567: ... --> URL: #39092 Reported by: user202729 Reviewer(s): Julian Rüth
2 parents cc231ef + e9dcaf2 commit a89c57b

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

src/sage/modular/arithgroup/farey_symbol.pyx

-1
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@ from sage.structure.richcmp cimport richcmp_not_equal
4747

4848
cdef extern from "sl2z.hpp":
4949
cppclass cpp_SL2Z "SL2Z":
50-
mpz_class a, b, c, d
5150
cpp_SL2Z(int, int, int, int)
5251
cpp_SL2Z(mpz_class, mpz_class, mpz_class, mpz_class)
5352
mpz_class a()

0 commit comments

Comments
 (0)