tests/ui/needless_borrow.rs

   1 #![feature(tool_lints)]
   2
   3 use std::borrow::Cow;
   4
   5 #[allow(clippy::trivially_copy_pass_by_ref)]
   6 fn x(y: &i32) -> i32 {
   7     *y
   8 }
   9
  10 #[warn(clippy::all, clippy::needless_borrow)]
  11 #[allow(unused_variables)]
  12 fn main() {
  13     let a = 5;
  14     let b = x(&a);
  15     let c = x(&&a);
  16     let s = &String::from("hi");
  17     let s_ident = f(&s); // should not error, because `&String` implements Copy, but `String` does not
  18     let g_val = g(&Vec::new()); // should not error, because `&Vec<T>` derefs to `&[T]`
  19     let vec = Vec::new();
  20     let vec_val = g(&vec); // should not error, because `&Vec<T>` derefs to `&[T]`
  21     h(&"foo"); // should not error, because the `&&str` is required, due to `&Trait`
  22     if let Some(ref cake) = Some(&5) {}
  23     let garbl = match 42 {
  24         44 => &a,
  25         45 => {
  26             println!("foo");
  27             &&a // FIXME: this should lint, too
  28         },
  29         46 => &&a,
  30         _ => panic!(),
  31     };
  32 }
  33
  34 fn f<T:Copy>(y: &T) -> T {
  35     *y
  36 }
  37
  38 fn g(y: &[u8]) -> u8 {
  39     y[0]
  40 }
  41
  42 trait Trait {}
  43
  44 impl<'a> Trait for &'a str {}
  45
  46 fn h(_: &Trait) {}
  47 #[warn(clippy::needless_borrow)]
  48 #[allow(dead_code)]
  49 fn issue_1432() {
  50     let mut v = Vec::<String>::new();
  51     let _ = v.iter_mut().filter(|&ref a| a.is_empty());
  52     let _ = v.iter().filter(|&ref a| a.is_empty());
  53
  54     let _ = v.iter().filter(|&a| a.is_empty());
  55 }