#20998: convert a few cmp to richcmp in some pyx files
-------------------------+-------------------------------------------------
       Reporter:         |        Owner:
  chapoton               |
           Type:         |       Status:  needs_work
  enhancement            |
       Priority:  major  |    Milestone:  sage-7.3
      Component:         |   Resolution:
  python3                |
       Keywords:         |    Merged in:
        Authors:         |    Reviewers:
  Frédéric Chapoton      |
Report Upstream:  N/A    |  Work issues:
         Branch:         |       Commit:
  public/20998           |  f4d65c76e96648ce0b1e0bd17122a3eec6bce18f
   Dependencies:         |     Stopgaps:
-------------------------+-------------------------------------------------
Changes (by vdelecroix):

 * status:  needs_review => needs_work


Comment:

 `_cmp_` (one underscore) and `__richcmp__` (two underscores) do not have
 the same semantic. In your code it seems that you assume that `self` and
 `other` have the same type and parent... which might not be the case.

--
Ticket URL: <https://trac.sagemath.org/ticket/20998#comment:3>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to