an-cabal / an-rope

an rope data structure
https://docs.rs/an-rope
MIT License
11 stars 2 forks source link

Avoid hidden copies #2

Closed deepinthebuild closed 7 years ago

deepinthebuild commented 7 years ago

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.