mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Auto merge of #9441 - Jarcho:hang_9433, r=Alexendoo

Browse Source 
Fix hang in `vec_init_then_push`

fixes #9433
changelog: Fix infinite loop in `vec_init_then_push`
This commit is contained in:
bors 2022-09-07 19:43:00 +00:00
commit da29f8928d
3 changed files with 15 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
clippy_lints/src/vec_init_then_push.rs
View File

@ -86,7 +86,7 @@ impl VecPushSearcher {
},
ExprKind::Unary(UnOp::Deref, _) | ExprKind::Index(..) if !needs_mut => {
let mut last_place = parent;
while let Some(parent) = get_parent_expr(cx, parent) {
while let Some(parent) = get_parent_expr(cx, last_place) {
if matches!(parent.kind, ExprKind::Unary(UnOp::Deref, _) | ExprKind::Field(..))
|| matches!(parent.kind, ExprKind::Index(e, _) if e.hir_id == last_place.hir_id)
{

6
tests/ui/vec_init_then_push.rs
View File

@ -104,3 +104,9 @@ fn _cond_push_with_large_start(x: bool) -> Vec<u32> {
v2
}
fn f() {
let mut v = Vec::new();
v.push((0i32, 0i32));
let y = v[0].0.abs();
}

9
tests/ui/vec_init_then_push.stderr
View File

@ -62,5 +62,12 @@ LL | | v2.push(1);
LL | | v2.push(0);
| |_______________^ help: consider using the `vec![]` macro: `let mut v2 = vec![..];`
error: aborting due to 7 previous errors
error: calls to `push` immediately after creation
--> $DIR/vec_init_then_push.rs:109:5
|
LL | / let mut v = Vec::new();
LL | | v.push((0i32, 0i32));
| |_________________________^ help: consider using the `vec![]` macro: `let v = vec![..];`
error: aborting due to 8 previous errors