New lint: Prefer mem::take over Vec::split_off(0)

[tesuji]

let mut vec = vec![1,2,3];
let vec2 = vec.split_off(0);

Should be written as

let vec2  = std::mem::take(&mut vec);

[tesuji]

Oh, wait. The semantics between two functions are different.
Vec::split_off leave the caller with same capacity while mem::take leave call with default value
(in this case an empty Vec).

[basil-cow]

I think vec.drain(..).collect() better represents the intention behind vec.split_off(0) (assuming it doesn't change capacity, which I don't think it is but seems like it's not specified either).

[andre-digiexam]

@rustbot claim