mirror of
https://github.com/rust-lang/rust-clippy
synced 2024-11-15 09:27:25 +00:00
f98ffa271d
Don't emit a lint when `pushed_item` was declared as mutable variable.
97 lines
2 KiB
Rust
97 lines
2 KiB
Rust
#![warn(clippy::same_item_push)]
|
|
|
|
fn mutate_increment(x: &mut u8) -> u8 {
|
|
*x += 1;
|
|
*x
|
|
}
|
|
|
|
fn increment(x: u8) -> u8 {
|
|
x + 1
|
|
}
|
|
|
|
fn main() {
|
|
// Test for basic case
|
|
let mut spaces = Vec::with_capacity(10);
|
|
for _ in 0..10 {
|
|
spaces.push(vec![b' ']);
|
|
}
|
|
|
|
let mut vec2: Vec<u8> = Vec::new();
|
|
let item = 2;
|
|
for _ in 5..=20 {
|
|
vec2.push(item);
|
|
}
|
|
|
|
let mut vec3: Vec<u8> = Vec::new();
|
|
for _ in 0..15 {
|
|
let item = 2;
|
|
vec3.push(item);
|
|
}
|
|
|
|
let mut vec4: Vec<u8> = Vec::new();
|
|
for _ in 0..15 {
|
|
vec4.push(13);
|
|
}
|
|
|
|
// Suggestion should not be given as pushed variable can mutate
|
|
let mut vec5: Vec<u8> = Vec::new();
|
|
let mut item: u8 = 2;
|
|
for _ in 0..30 {
|
|
vec5.push(mutate_increment(&mut item));
|
|
}
|
|
|
|
let mut vec6: Vec<u8> = Vec::new();
|
|
let mut item: u8 = 2;
|
|
let mut item2 = &mut mutate_increment(&mut item);
|
|
for _ in 0..30 {
|
|
vec6.push(mutate_increment(item2));
|
|
}
|
|
|
|
let mut vec7: Vec<usize> = Vec::new();
|
|
for (a, b) in [0, 1, 4, 9, 16].iter().enumerate() {
|
|
vec7.push(a);
|
|
}
|
|
|
|
let mut vec8: Vec<u8> = Vec::new();
|
|
for i in 0..30 {
|
|
vec8.push(increment(i));
|
|
}
|
|
|
|
let mut vec9: Vec<u8> = Vec::new();
|
|
for i in 0..30 {
|
|
vec9.push(i + i * i);
|
|
}
|
|
|
|
// Suggestion should not be given as there are multiple pushes that are not the same
|
|
let mut vec10: Vec<u8> = Vec::new();
|
|
let item: u8 = 2;
|
|
for _ in 0..30 {
|
|
vec10.push(item);
|
|
vec10.push(item * 2);
|
|
}
|
|
|
|
// Suggestion should not be given as Vec is not involved
|
|
for _ in 0..5 {
|
|
println!("Same Item Push");
|
|
}
|
|
|
|
struct A {
|
|
kind: u32,
|
|
}
|
|
let mut vec_a: Vec<A> = Vec::new();
|
|
for i in 0..30 {
|
|
vec_a.push(A { kind: i });
|
|
}
|
|
let mut vec12: Vec<u8> = Vec::new();
|
|
for a in vec_a {
|
|
vec12.push(2u8.pow(a.kind));
|
|
}
|
|
|
|
// Fix #5902
|
|
let mut vec13: Vec<u8> = Vec::new();
|
|
let mut item = 0;
|
|
for _ in 0..10 {
|
|
vec13.push(item);
|
|
item += 10;
|
|
}
|
|
}
|