#17726: Replace PY_TYPE() by type()
-------------------------------------+-------------------------------------
Reporter: jdemeyer | Owner:
Type: enhancement | Status: positive_review
Priority: major | Milestone: sage-6.5
Component: cython | Resolution:
Keywords: | Merged in:
Authors: Jeroen Demeyer | Reviewers: Marc Mezzarobba
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jdemeyer/ticket/17726 | 05e0633a1e366dacf7337b2f4895daf137b38bad
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by jdemeyer):
Replying to [comment:5 mmezzarobba]:
> Replying to [comment:3 vdelecroix]:
> > Some of them are already in #17727. Could we set the latter as a
dependency and completely get rid of `PY_TYPE`?
>
> With #17668 + #17725 + #17726 + #17727, the only remaining occurrences
of `PY_TYPE` are in `stdsage.*`.
Indeed, I've made these different ticket independent for easier reviewing.
If all four of the above tickets are merged, I plan to make a ticket to
remove `PY_TYPE_CHECK` and another one to remove the `stdsage.pxi`
includes.
I would keep the functions in `stdsage.pxi` for now, just to support
existing code on Trac which uses them (consider them "deprecated").
--
Ticket URL: <http://trac.sagemath.org/ticket/17726#comment:6>
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.
