Make VecModel::remove() return the removed value like Vec

[lukas-jung]

VecModel's remove() currently just removes the element but does not return it. VecModel should behave similar to Vec so we should change this behavior.

[tronical]

I'm curious, why is this a breaking change?

[ogoffart]

If you change the return value of a function, it is a breaking change.

fn remove(model: &mut VecModel) {
    model.remove(42) // note: no semi colon
}

[tronical]

Thanks, makes sense :)