Avoid hidden copies

[deepinthebuild]

Currently Rope::merge takes a reference and makes a copy. It should take the second argument by value. If a caller wants to pass in a copy they can explicitly make a copy and pass that in.