]> git.lizzy.rs Git - rust.git/blob - tests/ui/option_map_unwrap_or.stderr
Auto merge of #5525 - flip1995:issue_1654, r=phansch
[rust.git] / tests / ui / option_map_unwrap_or.stderr
1 error: called `map(f).unwrap_or(a)` on an `Option` value. This can be done more directly by calling `map_or(a, f)` instead
2   --> $DIR/option_map_unwrap_or.rs:20:13
3    |
4 LL |       let _ = opt.map(|x| x + 1)
5    |  _____________^
6 LL | |         // Should lint even though this call is on a separate line.
7 LL | |         .unwrap_or(0);
8    | |_____________________^
9    |
10    = note: `-D clippy::option-map-unwrap-or` implied by `-D warnings`
11 help: use `map_or(a, f)` instead
12    |
13 LL |     let _ = opt.map_or(0, |x| x + 1);
14    |                 ^^^^^^ ^^          --
15
16 error: called `map(f).unwrap_or(a)` on an `Option` value. This can be done more directly by calling `map_or(a, f)` instead
17   --> $DIR/option_map_unwrap_or.rs:24:13
18    |
19 LL |       let _ = opt.map(|x| {
20    |  _____________^
21 LL | |         x + 1
22 LL | |     }
23 LL | |     ).unwrap_or(0);
24    | |__________________^
25    |
26 help: use `map_or(a, f)` instead
27    |
28 LL |     let _ = opt.map_or(0, |x| {
29 LL |         x + 1
30 LL |     }
31 LL |     );
32    |
33
34 error: called `map(f).unwrap_or(a)` on an `Option` value. This can be done more directly by calling `map_or(a, f)` instead
35   --> $DIR/option_map_unwrap_or.rs:28:13
36    |
37 LL |       let _ = opt.map(|x| x + 1)
38    |  _____________^
39 LL | |         .unwrap_or({
40 LL | |             0
41 LL | |         });
42    | |__________^
43    |
44 help: use `map_or(a, f)` instead
45    |
46 LL |     let _ = opt.map_or({
47 LL |             0
48 LL |         }, |x| x + 1);
49    |
50
51 error: called `map(f).unwrap_or(None)` on an `Option` value. This can be done more directly by calling `and_then(f)` instead
52   --> $DIR/option_map_unwrap_or.rs:33:13
53    |
54 LL |     let _ = opt.map(|x| Some(x + 1)).unwrap_or(None);
55    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
56    |
57 help: use `and_then(f)` instead
58    |
59 LL |     let _ = opt.and_then(|x| Some(x + 1));
60    |                 ^^^^^^^^                --
61
62 error: called `map(f).unwrap_or(None)` on an `Option` value. This can be done more directly by calling `and_then(f)` instead
63   --> $DIR/option_map_unwrap_or.rs:35:13
64    |
65 LL |       let _ = opt.map(|x| {
66    |  _____________^
67 LL | |         Some(x + 1)
68 LL | |     }
69 LL | |     ).unwrap_or(None);
70    | |_____________________^
71    |
72 help: use `and_then(f)` instead
73    |
74 LL |     let _ = opt.and_then(|x| {
75 LL |         Some(x + 1)
76 LL |     }
77 LL |     );
78    |
79
80 error: called `map(f).unwrap_or(None)` on an `Option` value. This can be done more directly by calling `and_then(f)` instead
81   --> $DIR/option_map_unwrap_or.rs:39:13
82    |
83 LL |       let _ = opt
84    |  _____________^
85 LL | |         .map(|x| Some(x + 1))
86 LL | |         .unwrap_or(None);
87    | |________________________^
88    |
89 help: use `and_then(f)` instead
90    |
91 LL |         .and_then(|x| Some(x + 1));
92    |          ^^^^^^^^                --
93
94 error: called `map(f).unwrap_or(a)` on an `Option` value. This can be done more directly by calling `map_or(a, f)` instead
95   --> $DIR/option_map_unwrap_or.rs:50:13
96    |
97 LL |     let _ = Some("prefix").map(|p| format!("{}.", p)).unwrap_or(id);
98    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
99    |
100 help: use `map_or(a, f)` instead
101    |
102 LL |     let _ = Some("prefix").map_or(id, |p| format!("{}.", p));
103    |                            ^^^^^^ ^^^                      --
104
105 error: called `map(f).unwrap_or_else(g)` on an `Option` value. This can be done more directly by calling `map_or_else(g, f)` instead
106   --> $DIR/option_map_unwrap_or.rs:54:13
107    |
108 LL |       let _ = opt.map(|x| x + 1)
109    |  _____________^
110 LL | |         // Should lint even though this call is on a separate line.
111 LL | |         .unwrap_or_else(|| 0);
112    | |_____________________________^
113    |
114    = note: `-D clippy::option-map-unwrap-or-else` implied by `-D warnings`
115    = note: replace `map(|x| x + 1).unwrap_or_else(|| 0)` with `map_or_else(|| 0, |x| x + 1)`
116
117 error: called `map(f).unwrap_or_else(g)` on an `Option` value. This can be done more directly by calling `map_or_else(g, f)` instead
118   --> $DIR/option_map_unwrap_or.rs:58:13
119    |
120 LL |       let _ = opt.map(|x| {
121    |  _____________^
122 LL | |         x + 1
123 LL | |     }
124 LL | |     ).unwrap_or_else(|| 0);
125    | |__________________________^
126
127 error: called `map(f).unwrap_or_else(g)` on an `Option` value. This can be done more directly by calling `map_or_else(g, f)` instead
128   --> $DIR/option_map_unwrap_or.rs:62:13
129    |
130 LL |       let _ = opt.map(|x| x + 1)
131    |  _____________^
132 LL | |         .unwrap_or_else(||
133 LL | |             0
134 LL | |         );
135    | |_________^
136
137 error: aborting due to 10 previous errors
138