Clippy lints / Performance
Clippy lint · Performance · checked with Clippy 0.1.99
clippy::extend_with_drain: use of `extend` instead of `append` for adding the full range of a second vector
What it meansChecks for occurrences where one vector gets extended instead of append It is in the performance group and warns by default.
What you see
warning: use of `extend` instead of `append` for adding the full range of a second vector --> main.rs:6:1 | 6 | a.extend(b.drain(..)); | ^^^^^^^^^^^^^^^^^^^^^ help: try: `a.append(&mut b)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/stable/index.html#extend_with_drain = note: `#[warn(clippy::extend_with_drain)]` on by default
Real output of Clippy 0.1.99 (Rust 1.99) on the example below. Names and line numbers in yours will differ.
Does it matter?
Using append instead of extend is more concise and faster
Code that could be written to run faster. Warns by default.
How to fix it
Clippy can fix this one for you: run cargo clippy --fix (add --allow-dirty if you have uncommitted changes). Review the diff before you commit.
Flagged
let mut a = vec![1, 2, 3];
let mut b = vec![4, 5, 6];
a.extend(b.drain(..));
Write this instead
let mut a = vec![1, 2, 3];
let mut b = vec![4, 5, 6];
a.append(&mut b);
Turn it off, or on
Allow it where it does not fit your code, as narrowly as you can:
On one item
#[allow(clippy::extend_with_drain)]
fn my_function() { /* ... */ }Whole crate (Cargo.toml)
[lints.clippy]
extend_with_drain = "allow"One run
cargo clippy -- -A clippy::extend-with-drainThe [lints] table needs Rust 1.74 or newer. In a workspace, put it under [workspace.lints.clippy] and add [lints] workspace = true to each member.
Clippy's own documentation
What it does
Checks for occurrences where one vector gets extended instead of append
Why is this bad?
Using append instead of extend is more concise and faster
Example
let mut a = vec![1, 2, 3];
let mut b = vec![4, 5, 6];
a.extend(b.drain(..));
Use instead:
let mut a = vec![1, 2, 3];
let mut b = vec![4, 5, 6];
a.append(&mut b);
From the Clippy lint list (0.1.99), part of the Rust project, MIT or Apache-2.0. Official entry.
Related lints
- large_enum_variantlarge size difference between variants
- double_ended_iterator_lastcalled `Iterator::last` on a `DoubleEndedIterator`; this will needlessly iterate the entire iterator
- drain_collectdraining all elements of a collection into a new collection of the same type
- expect_fun_callChecks for calls to .expect(&format!(...)), .expect(foo(..)), etc., and suggests to use unwrap_or_else instead
- format_in_format_argsDetects format! within the arguments of another macro that does formatting such as format! itself, write! or println!
- iter_overeager_clonedChecks for usage of _.cloned().<func>() where call to .cloned() can be postponed
- large_const_arraysChecks for large const arrays that should be defined as static instead
All 847 Clippy lints · Rust compiler errors · Cargo.toml checker
Up next
Related lintclippy::large_enum_variantlarge size difference between variantsRead it next →Did this answer your question?