Ticket #6809 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

abstract_methods_of_class

Reported by: nthiery Owned by: nthiery
Priority: major Milestone: sage-4.2
Component: misc Keywords: abstract methods
Cc: sage-combinat Work issues:
Report Upstream: Reviewers: Florent Hivert
Authors: Nicolas M. Thiéry Merged in: sage-4.2.alpha1
Dependencies: Stopgaps:

Description

Implement a utility abstract_methods_of_class which lists all the optional and mandatory abstract methods of the class.

Attachments

trac_6809_abstract_methods_of_class.patch Download (3.4 KB) - added by nthiery 4 years ago.
trac_6809_abstract_methods_of_class.2.patch Download (3.9 KB) - added by hivert 4 years ago.
New version with is_optional method
trac_6809_abstract_methods_of_class.3.patch Download (3.9 KB) - added by hivert 4 years ago.
Added missing blank line

Change History

Changed 4 years ago by nthiery

comment:1 Changed 4 years ago by nthiery

Changed 4 years ago by hivert

New version with is_optional method

comment:2 Changed 4 years ago by hivert

  • Status changed from new to needs_review
  • Reviewers changed from Franco Saliola? to Florent Hivert

I'm done reviewing this patch. It is good upto a small detail: the code access to a private attribute. I've added an accessor method.

I just uploaded a new version of the patch after review which add a is_optional method for better encapsulation (as suggested by former comment in the code).

Nicolas: please add a positive review when you finished reviewing my changes.

Cheers,

Florent

comment:3 Changed 4 years ago by hivert

During his review of my change Nicolas spotted a missing blank line after 'EXAMPLE::'... I'm re-uploading the patch...

Cheers,

Florent

Changed 4 years ago by hivert

Added missing blank line

comment:4 Changed 4 years ago by hivert

  • Status changed from needs_review to positive_review

After Blank line added, Nicolas allows me to put the positive review.

Only apply the last trac_6809_abstract_methods_of_class.3.patch

Florent

comment:5 Changed 4 years ago by mhansen

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-4.2.alpha1
Note: See TracTickets for help on using tickets.