Changes between Version 1 and Version 3 of Ticket #22611


Ignore:
Timestamp:
03/17/17 21:46:32 (5 years ago)
Author:
jdemeyer
Comment:

New commits:

3264657Replace _sage_doc_ by a __doc__ descriptor

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #22611

    • Property Status changed from new to needs_review
    • Property Cc hivert nthiery added
    • Property Branch changed from to u/jdemeyer/ticket/22611
    • Property Commit changed from to 326465738e5a563ebffc86196f400829650cee63
  • Ticket #22611 – Description

    v1 v3  
    11The method `_sage_doc_` is used to have custom documentation for instances of a class. Such a use case makes sense, but it should be implemented as the `__doc__` attribute instead of the Sage-specific `_sage_doc_`.
    22
    3 This is easy for Python classes, but it might be problematic for `cdef class`es.
    4 
    5 Unfortunately, this hits some Cython bugs involving @decorators.
     3I tried various ways of implementing this and I was various Cython bugs along the way. The current solution is not the cleanest, but it works in all cases.