#13192: some code clean up for sage/graphs/graph.py
----------------------------------+-----------------------------------------
Reporter: eisermbi | Owner: jason, ncohen, rlm
Type: defect | Status: needs_review
Priority: major | Milestone: sage-5.2
Component: graph theory | Resolution:
Keywords: sparse6_string | Work issues:
Report Upstream: N/A | Reviewers: Nathann Cohen
Authors: Birk Eisermann | Merged in:
Dependencies: | Stopgaps:
----------------------------------+-----------------------------------------
Changes (by eisermbi):
* status: needs_info => needs_review
Old description:
> The function compare_edges() of module 'sage.graphs.graph.py' is used
> only in one place, function sparse6_string(), and it works only on graphs
> whose vertices can be compared with the < operator, e.g. integers.
> Furthermore, the documentation is incomplete.
>
> Hence, suggesting to make it a local function at the place where it is
> used.
New description:
The function compare_edges() of module 'sage.graphs.graph.py' is a simple
comparison function used only in function sparse6_string(), and it works
only on graphs whose vertices can be compared with the < operator, e.g.
integers. Furthermore, the documentation is incomplete.
Suggesting to replace it by a built-in function at the place where it was
used.
--
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13192#comment:6>
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.
