#16580: FiniteStateMachine: cleanup code
-------------------------------------+-------------------------------------
Reporter: cheuberg | Owner:
Type: defect | Status: new
Priority: minor | Milestone: sage-6.3
Component: finite state machines | Keywords:
Merged in: | finite_state_machine
Reviewers: | Authors: Clemens Heuberger
Work issues: | Report Upstream: N/A
Commit: | Branch: u/cheuberg/fsm
fd618dcb88bf9771419caf8d86ce617b644e6e54| /code-cleanup-post-16067
Stopgaps: | Dependencies:
-------------------------------------+-------------------------------------
#16067 introduced some changes into the finite state machine code by
rewritting several instances of filter into list comprehensions. When re-
reading the resulting code, I had the impression that the code can be
simplified somewhat by combining it with the next lines (two occurrences).
So I simplified these two occurrences.
Another occurrence touched very inefficient code in {{{_latex_}}} which
was scheduled to be rewritten at some state anyway. So this is now done.
Unfortunately, this changes the order of the transitions in the latex
output, so the output in a few corresponding doctests had to be reordered,
too.
Furthermore, when #15990 rewrote the raise statement for Python 3
compatibility, one occurrence of the old syntax was re-introduced in a
separate branch. Fixed this.
Finally, added two doctests where I wanted to make sure that two methods
work as promised.
--
Ticket URL: <http://trac.sagemath.org/ticket/16580>
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.
