#15290: BasisExchangeMatroid.groundset_list() should return a copy
-------------------------------------+-------------------------------------
Reporter: Stefan | Owner:
Type: defect | Status: new
Priority: major | Milestone: sage-6.1
Component: matroid theory | Resolution:
Keywords: | Merged in:
Authors: | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/Rudi/ticket/15290 | 86a82fa954d37d39ab19f619aa07fbb860ebc17f
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by Rudi):
* commit: => 86a82fa954d37d39ab19f619aa07fbb860ebc17f
Comment:
I implemented solution 2, and changed BasisExchangeMatroid._E to a tuple
instead of a list. It was my beginner's error not to use the immutable
tuple here in the first place.
I did not notice any effects on the efficiency of the code.
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=d6047de30b7d53f7b0e35cece45c105dc91347fc
d6047de]||{{{Changed type of BasisExchangeMatroid._E and
SetSystem._groundset from `list' to `tuple'.}}}||
||[http://git.sagemath.org/sage.git/commit/?id=86a82fa954d37d39ab19f619aa07fbb860ebc17f
86a82fa]||{{{Changed docstring of BasisExchangeMatroid.groundset_list() to
include a test verifying that ticket 15290 was fixed.}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/15290#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 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/groups/opt_out.
