#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:
-------------------------------------+-------------------------------------
Changes (by mmezzarobba):
* status: needs_info => positive_review
* reviewer: => Marc Mezzarobba
Comment:
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.*`.
And the earlier in the release cycle patches of this kind get merged, the
better. So if you don't mind let's leave the issue of removing `PY_TYPE`
itself for later, as the present ticket makes sense without it.
--
Ticket URL: <http://trac.sagemath.org/ticket/17726#comment:5>
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.
