Graph.relabel modifies the dictionary given as an input

[nathanncohen]

Well, it's as the title says...

The patch just fixes it in the obvious way, and moves some code around to make it easier to read.

(reported by Nicolas Thiéry)

sage: g = Graph({1:[2,3]})
sage: rel = {1:'a', 2:'b'}
sage: g.relabel(rel)
sage: g.vertices()
[3, 'a', 'b']
sage: rel
{1: 'a', 2: 'b', 3: 3}

Nathann

CC: @nthiery

Component: graph theory

Keywords: Cernay2012

Author: Nathann Cohen

Reviewer: Nicolas M. Thiéry

Merged: sage-5.0.beta5

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

[nathanncohen]

Changed keywords from none to Cernay2012

[nthiery]

Reviewer: Nicolas M. Thiéry

[nthiery]

comment:2

Attachment: trac_12477.patch.gz

[nthiery]

comment:3

Discussed together. The tests on the file pass. Positive review assuming the buildbot gives a green light.

Thanks Nathann!

[jdemeyer]

Merged: sage-5.0.beta5