2 // aux-build: proc_macro_with_span.rs
3 #![warn(clippy::unnecessary_lazy_evaluations)]
4 #![allow(clippy::redundant_closure)]
5 #![allow(clippy::bind_instead_of_map)]
6 #![allow(clippy::map_identity)]
8 extern crate proc_macro_with_span;
9 use proc_macro_with_span::with_span;
11 struct Deep(Option<usize>);
13 #[derive(Copy, Clone)]
19 fn return_some_field(&self) -> usize {
24 fn some_call<T: Default>() -> T {
28 struct Issue9427(i32);
30 impl Drop for Issue9427 {
32 println!("{}", self.0);
37 let astronomers_pi = 10;
38 let ext_arr: [usize; 1] = [2];
39 let ext_str = SomeStruct { some_field: 10 };
41 let mut opt = Some(42);
42 let ext_opt = Some(42);
43 let nested_opt = Some(Some(42));
44 let nested_tuple_opt = Some(Some((42, 43)));
47 // Should lint - Option
48 let _ = opt.unwrap_or_else(|| 2);
49 let _ = opt.unwrap_or_else(|| astronomers_pi);
50 let _ = opt.unwrap_or_else(|| ext_str.some_field);
51 let _ = opt.unwrap_or_else(|| ext_arr[0]);
52 let _ = opt.and_then(|_| ext_opt);
53 let _ = opt.or_else(|| ext_opt);
54 let _ = opt.or_else(|| None);
55 let _ = opt.get_or_insert_with(|| 2);
56 let _ = opt.ok_or_else(|| 2);
57 let _ = nested_tuple_opt.unwrap_or_else(|| Some((1, 2)));
58 let _ = cond.then(|| astronomers_pi);
60 // Cases when unwrap is not called on a simple variable
61 let _ = Some(10).unwrap_or_else(|| 2);
62 let _ = Some(10).and_then(|_| ext_opt);
63 let _: Option<usize> = None.or_else(|| ext_opt);
64 let _ = None.get_or_insert_with(|| 2);
65 let _: Result<usize, usize> = None.ok_or_else(|| 2);
66 let _: Option<usize> = None.or_else(|| None);
68 let mut deep = Deep(Some(42));
69 let _ = deep.0.unwrap_or_else(|| 2);
70 let _ = deep.0.and_then(|_| ext_opt);
71 let _ = deep.0.or_else(|| None);
72 let _ = deep.0.get_or_insert_with(|| 2);
73 let _ = deep.0.ok_or_else(|| 2);
75 // Should not lint - Option
76 let _ = opt.unwrap_or_else(|| ext_str.return_some_field());
77 let _ = nested_opt.unwrap_or_else(|| Some(some_call()));
78 let _ = nested_tuple_opt.unwrap_or_else(|| Some((some_call(), some_call())));
79 let _ = opt.or_else(some_call);
80 let _ = opt.or_else(|| some_call());
81 let _: Result<usize, usize> = opt.ok_or_else(|| some_call());
82 let _: Result<usize, usize> = opt.ok_or_else(some_call);
83 let _ = deep.0.get_or_insert_with(|| some_call());
84 let _ = deep.0.or_else(some_call);
85 let _ = deep.0.or_else(|| some_call());
86 let _ = opt.ok_or_else(|| ext_arr[0]);
88 // Should not lint - bool
89 let _ = (0 == 1).then(|| Issue9427(0)); // Issue9427 has a significant drop
91 // should not lint, bind_instead_of_map takes priority
92 let _ = Some(10).and_then(|idx| Some(ext_arr[idx]));
93 let _ = Some(10).and_then(|idx| Some(idx));
95 // should lint, bind_instead_of_map doesn't apply
96 let _: Option<usize> = None.or_else(|| Some(3));
97 let _ = deep.0.or_else(|| Some(3));
98 let _ = opt.or_else(|| Some(3));
100 // Should lint - Result
101 let res: Result<usize, usize> = Err(5);
102 let res2: Result<usize, SomeStruct> = Err(SomeStruct { some_field: 5 });
104 let _ = res2.unwrap_or_else(|_| 2);
105 let _ = res2.unwrap_or_else(|_| astronomers_pi);
106 let _ = res2.unwrap_or_else(|_| ext_str.some_field);
108 // Should not lint - Result
109 let _ = res.unwrap_or_else(|err| err);
110 let _ = res.unwrap_or_else(|err| ext_arr[err]);
111 let _ = res2.unwrap_or_else(|err| err.some_field);
112 let _ = res2.unwrap_or_else(|err| err.return_some_field());
113 let _ = res2.unwrap_or_else(|_| ext_str.return_some_field());
115 // should not lint, bind_instead_of_map takes priority
116 let _: Result<usize, usize> = res.and_then(|x| Ok(x));
117 let _: Result<usize, usize> = res.or_else(|err| Err(err));
119 let _: Result<usize, usize> = res.and_then(|_| Ok(2));
120 let _: Result<usize, usize> = res.and_then(|_| Ok(astronomers_pi));
121 let _: Result<usize, usize> = res.and_then(|_| Ok(ext_str.some_field));
123 let _: Result<usize, usize> = res.or_else(|_| Err(2));
124 let _: Result<usize, usize> = res.or_else(|_| Err(astronomers_pi));
125 let _: Result<usize, usize> = res.or_else(|_| Err(ext_str.some_field));
127 // should lint, bind_instead_of_map doesn't apply
128 let _: Result<usize, usize> = res.and_then(|_| Err(2));
129 let _: Result<usize, usize> = res.and_then(|_| Err(astronomers_pi));
130 let _: Result<usize, usize> = res.and_then(|_| Err(ext_str.some_field));
132 let _: Result<usize, usize> = res.or_else(|_| Ok(2));
133 let _: Result<usize, usize> = res.or_else(|_| Ok(astronomers_pi));
134 let _: Result<usize, usize> = res.or_else(|_| Ok(ext_str.some_field));
135 let _: Result<usize, usize> = res.
142 or_else(|_| Ok(ext_str.some_field));
144 // neither bind_instead_of_map nor unnecessary_lazy_eval applies here
145 let _: Result<usize, usize> = res.and_then(|x| Err(x));
146 let _: Result<usize, usize> = res.or_else(|err| Ok(err));
151 // should not lint, is in proc macro
152 with_span!(span Some(42).unwrap_or_else(|| 2););