#18121: Patch Cython with PyTypeObject members
-------------------------------------+-------------------------------------
       Reporter:  jdemeyer           |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.6
      Component:  packages:          |   Resolution:
  standard                           |    Merged in:
       Keywords:  sd66               |    Reviewers:
        Authors:  Jeroen Demeyer     |  Work issues:
Report Upstream:  Fixed upstream,    |       Commit:
  but not in a stable release.       |  a10dcbaaaefb87d48cf337be367cd48f11ed6edf
         Branch:                     |     Stopgaps:
  u/jdemeyer/patch_cython_with_pytypeobject_members|
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by jdemeyer):

 Replying to [comment:20 nbruin]:
 > Replying to [comment:19 vdelecroix]:
 > > Why `void Py_VISIT(PyObject *o)` is not compatible with Cython? I do
 not quite understand what is this `__pyx_v_visit`. Is it because of Cython
 renaming arguments? If it is so, I do not find it very safe.
 >
 > Correct, and I agree. Depending on the user giving parameters a
 particular name ''and'' cython mangling the names in a particular way is
 more fragile than what the CPython macro does.
 How about generalizing the macro to accept `visit` and `arg` arguments?

 > We only use this stuff twice
 8 times if I count correctly.

--
Ticket URL: <http://trac.sagemath.org/ticket/18121#comment:21>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to