Implement Extend<&'a T> for T: Copy #13

interruptinuse · 2022-01-16T11:10:49Z

This PR supports the following Vec idiom:

let mut v: Vec<u8> = vec![];
v.extend("data".as_bytes());
assert_eq!(v, [0x64, 0x61, 0x74, 0x61]);

This supports the "extend with slice" idiom (see test_segvec_extend_ref).

mccolljr · 2022-01-16T22:13:45Z

Hey, thanks for the PR. This seems like a pretty straightforward change, and thanks for adding a test. Looks good to me.

mccolljr · 2022-01-16T22:18:39Z

Merged. When I'm back at my machine later I'll publish a new version

interruptinuse · 2022-01-17T10:27:19Z

Merged. When I'm back at my machine later I'll publish a new version

Sweet, may I squeeze #14 in as well before that?

Implement Extend<&'a T> for T: Copy

e5718a6

This supports the "extend with slice" idiom (see test_segvec_extend_ref).

mccolljr merged commit e397dce into mccolljr:master Jan 16, 2022

interruptinuse deleted the interruptinuse/extend-ref branch January 17, 2022 10:26

Provide feedback