bug in set_edge_label

[rlmill]

sage: G = Graph({0:{1:1}}, implementation='c_graph')
sage: G.num_edges()
1
sage: G.set_edge_label(0,1,1)
sage: G.num_edges()
2

Component: graph theory

Issue created by migration from https://trac.sagemath.org/ticket/3703

[jasongrout]

comment:1

Attachment: trac3703-set_edge_label.patch.gz

Positive review. Good catch!

Passes doctests in graphs/.py, graphs/.pyx, and graphs/base/*.pyx

[rlmill]

comment:2

I spent most of a day hunting for this one; I wasn't expecting to find it where I did. I was writing new code, so I was convinced it was there...

[jasongrout]

comment:3

That's a nice blog post you have talking about finding this bug:

http://rlmill.blogspot.com/2008/07/adinkras.html

[rlmill]

comment:5

This has been sitting positively reviewed for several release cycles now. What's going on??

[sagetrac-mabshoff]

comment:6

Replying to @rlmill:

This has been sitting positively reviewed for several release cycles now. What's going on??

3.0.5 was supposed to be ultra stable, but one bug introduced mandated another 3.0.6 stable release. So this patch fell by the side. It will be merged in 3.1.alpha0 in a couple minutes provided the doctests pass.

Cheers,

Michael

[sagetrac-mabshoff]

comment:7

Merged in Sage 3.1.alpha0