Have `Span` take self by value
Have `Span` take self by value

This was caught by Clippy. `Span` is an 8 byte copy type so it makes
sense to take it by value instead of reference. This is consistent with
what `merge` was already doing.

brendanzab/codespan