tests/ui/needless_borrow.rs

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