From 2fd671e4bd26e1c2ebef6fe3a0ce6194ffc14182 Mon Sep 17 00:00:00 2001 From: Michael Wright Date: Thu, 5 Apr 2018 17:59:35 +0200 Subject: [PATCH] Move ref cow tests This commit moves the ref cow tests from needless_borrow.rs to ptr_arg.rs where all the other PTR_ARG tests are. --- tests/ui/needless_borrow.rs | 9 --------- tests/ui/needless_borrow.stderr | 10 +--------- tests/ui/ptr_arg.rs | 12 ++++++++++-- tests/ui/ptr_arg.stderr | 8 +++++++- 4 files changed, 18 insertions(+), 21 deletions(-) diff --git a/tests/ui/needless_borrow.rs b/tests/ui/needless_borrow.rs index 088d33b87..99500cd07 100644 --- a/tests/ui/needless_borrow.rs +++ b/tests/ui/needless_borrow.rs @@ -51,12 +51,3 @@ fn issue_1432() { let _ = v.iter().filter(|&a| a.is_empty()); } - -#[allow(dead_code)] -fn test_cow_with_ref(c: &Cow<[i32]>) { -} - -#[allow(dead_code)] -fn test_cow(c: Cow<[i32]>) { - let _c = c; -} diff --git a/tests/ui/needless_borrow.stderr b/tests/ui/needless_borrow.stderr index e319efa93..fde38508b 100644 --- a/tests/ui/needless_borrow.stderr +++ b/tests/ui/needless_borrow.stderr @@ -38,13 +38,5 @@ error: this pattern creates a reference to a reference 50 | let _ = v.iter().filter(|&ref a| a.is_empty()); | ^^^^^ help: change this to: `a` -error: using a reference to `Cow` is not recommended. - --> $DIR/needless_borrow.rs:56:25 - | -56 | fn test_cow_with_ref(c: &Cow<[i32]>) { - | ^^^^^^^^^^^ help: change this to: `&[i32]` - | - = note: `-D ptr-arg` implied by `-D warnings` - -error: aborting due to 7 previous errors +error: aborting due to 6 previous errors diff --git a/tests/ui/ptr_arg.rs b/tests/ui/ptr_arg.rs index 14b26e168..ce572be7a 100644 --- a/tests/ui/ptr_arg.rs +++ b/tests/ui/ptr_arg.rs @@ -1,8 +1,8 @@ - - #![allow(unused, many_single_char_names)] #![warn(ptr_arg)] +use std::borrow::Cow; + fn do_vec(x: &Vec) { //Nothing here } @@ -67,3 +67,11 @@ fn false_positive_capacity_too(x: &String) -> String { x.clone() } +#[allow(dead_code)] +fn test_cow_with_ref(c: &Cow<[i32]>) { +} + +#[allow(dead_code)] +fn test_cow(c: Cow<[i32]>) { + let _c = c; +} diff --git a/tests/ui/ptr_arg.stderr b/tests/ui/ptr_arg.stderr index bf8608111..a29e393ba 100644 --- a/tests/ui/ptr_arg.stderr +++ b/tests/ui/ptr_arg.stderr @@ -76,5 +76,11 @@ help: change `y.as_str()` to 62 | let c = y; | ^ -error: aborting due to 6 previous errors +error: using a reference to `Cow` is not recommended. + --> $DIR/ptr_arg.rs:71:25 + | +71 | fn test_cow_with_ref(c: &Cow<[i32]>) { + | ^^^^^^^^^^^ help: change this to: `&[i32]` + +error: aborting due to 7 previous errors