Fix reserve over allocating underlying buffer

[schultetwin1]

Fixes calls to reserve when the underlying shared buffer was already
big enough to fit the requested capacity. Previously a new even larger
buffer was created anyways. This could eventually lead to an OOM
condition.

Fixes #559

[schultetwin1]

This appears to have been introduced by #555

[schultetwin1]

Here is an example of how you could cause an OOM error

#[test]
fn reserve_in_arc_unique_does_not_overallocate_after_multiple_splits() {
    let mut bytes = BytesMut::from(LONG);
    let orig_capacity = bytes.capacity();
    for _ in 0..100 {
        drop(bytes.split_off(LONG.len() / 2));

        // now bytes is Arc and refcount == 1

        let new_capacity = bytes.capacity();
        bytes.reserve(orig_capacity - new_capacity);
    }
    assert_eq!(bytes.capacity(), orig_capacity);
}

[Darksonn]

Thanks for pointing this out. We should be careful to ensure that there aren't any other cases we missed. With this logic we can show that if v_capacity >= 2*new_cap, then it will not reallocate. That seems sufficient to me.

[schultetwin1]

Thanks for the quick merge! And I see you are already prepping a new release (1.2.1), thank you!!