#12380: Move methods from Word_nfactor_enumerable to FiniteWord_class
-----------------------------+----------------------------------------------
Reporter: slabbe | Owner: slabbe
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.0
Component: combinatorics | Keywords:
Work_issues: | Upstream: N/A
Reviewer: | Author: Sébastien Labbé
Merged: | Dependencies: #9958
-----------------------------+----------------------------------------------
Changes (by newvalueoldvalue):
* cc: ablondin, vdelecroix (added)
* status: new => needs_review
* author: => Sébastien Labbé
Old description:
> The file {{{sage/combinat/words/nfactor_enumerable_word.py}}} and its
> class {{{Word_nfactor_enumerable}}} were introduced by ticket #8604. Just
> when it got merged into Sage, I realized it was a mistake. This class
> should not exist. The fact that its name was ugly was an indice. All
> those methods should be in a Language class (see #12225). It would get
> things a lot cleaner.
>
> Only the class {{{FiniteWord_class}}} was inheriting from it, so it farly
> easy to fix : just move the 14 methods from one class to the other.
New description:
The file {{{sage/combinat/words/nfactor_enumerable_word.py}}} and its
class {{{Word_nfactor_enumerable}}} were introduced by ticket #8604. Just
when it got merged into Sage, I realized it was a mistake. This class
should not exist. The fact that its name was ugly was an indice. All those
methods should be in a Language class (see #12225). It would get things a
lot cleaner.
Only the class {{{FiniteWord_class}}} was inheriting from it, so it farly
easy to fix : just move the 14 methods from one class to the other.
Note : this change will be unnoticed by the user.
--
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12380#comment:2>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
