tests/ui/methods.stderr

   1 error: defining a method called `add` on this type; consider implementing the `std::ops::Add` trait or choosing a less ambiguous name
   2   --> $DIR/methods.rs:37:5
   3    |
   4 LL |     pub fn add(self, other: T) -> T { self }
   5    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   6    |
   7    = note: `-D clippy::should-implement-trait` implied by `-D warnings`
   8
   9 error: methods called `into_*` usually take self by value; consider choosing a less ambiguous name
  10   --> $DIR/methods.rs:48:17
  11    |
  12 LL |     fn into_u16(&self) -> u16 { 0 }
  13    |                 ^^^^^
  14    |
  15    = note: `-D clippy::wrong-self-convention` implied by `-D warnings`
  16
  17 error: methods called `to_*` usually take self by reference; consider choosing a less ambiguous name
  18   --> $DIR/methods.rs:50:21
  19    |
  20 LL |     fn to_something(self) -> u32 { 0 }
  21    |                     ^^^^
  22
  23 error: methods called `new` usually take no self; consider choosing a less ambiguous name
  24   --> $DIR/methods.rs:52:12
  25    |
  26 LL |     fn new(self) -> Self { unimplemented!(); }
  27    |            ^^^^
  28
  29 error: called `map(f).unwrap_or(a)` on an Option value. This can be done more directly by calling `map_or(a, f)` instead
  30   --> $DIR/methods.rs:120:13
  31    |
  32 LL |       let _ = opt.map(|x| x + 1)
  33    |  _____________^
  34 LL | |
  35 LL | |                .unwrap_or(0); // should lint even though this call is on a separate line
  36    | |____________________________^
  37    |
  38    = note: `-D clippy::option-map-unwrap-or` implied by `-D warnings`
  39    = note: replace `map(|x| x + 1).unwrap_or(0)` with `map_or(0, |x| x + 1)`
  40
  41 error: called `map(f).unwrap_or(a)` on an Option value. This can be done more directly by calling `map_or(a, f)` instead
  42   --> $DIR/methods.rs:124:13
  43    |
  44 LL |       let _ = opt.map(|x| {
  45    |  _____________^
  46 LL | |                         x + 1
  47 LL | |                     }
  48 LL | |               ).unwrap_or(0);
  49    | |____________________________^
  50
  51 error: called `map(f).unwrap_or(a)` on an Option value. This can be done more directly by calling `map_or(a, f)` instead
  52   --> $DIR/methods.rs:128:13
  53    |
  54 LL |       let _ = opt.map(|x| x + 1)
  55    |  _____________^
  56 LL | |                .unwrap_or({
  57 LL | |                     0
  58 LL | |                 });
  59    | |__________________^
  60
  61 error: called `map(f).unwrap_or(None)` on an Option value. This can be done more directly by calling `and_then(f)` instead
  62   --> $DIR/methods.rs:133:13
  63    |
  64 LL |     let _ = opt.map(|x| Some(x + 1)).unwrap_or(None);
  65    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  66    |
  67    = note: replace `map(|x| Some(x + 1)).unwrap_or(None)` with `and_then(|x| Some(x + 1))`
  68
  69 error: called `map(f).unwrap_or(None)` on an Option value. This can be done more directly by calling `and_then(f)` instead
  70   --> $DIR/methods.rs:135:13
  71    |
  72 LL |       let _ = opt.map(|x| {
  73    |  _____________^
  74 LL | |         Some(x + 1)
  75 LL | |     }
  76 LL | |     ).unwrap_or(None);
  77    | |_____________________^
  78
  79 error: called `map(f).unwrap_or(None)` on an Option value. This can be done more directly by calling `and_then(f)` instead
  80   --> $DIR/methods.rs:139:13
  81    |
  82 LL |       let _ = opt
  83    |  _____________^
  84 LL | |         .map(|x| Some(x + 1))
  85 LL | |         .unwrap_or(None);
  86    | |________________________^
  87    |
  88    = note: replace `map(|x| Some(x + 1)).unwrap_or(None)` with `and_then(|x| Some(x + 1))`
  89
  90 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
  91   --> $DIR/methods.rs:147:13
  92    |
  93 LL |       let _ = opt.map(|x| x + 1)
  94    |  _____________^
  95 LL | |
  96 LL | |                .unwrap_or_else(|| 0); // should lint even though this call is on a separate line
  97    | |____________________________________^
  98    |
  99    = note: `-D clippy::option-map-unwrap-or-else` implied by `-D warnings`
 100    = note: replace `map(|x| x + 1).unwrap_or_else(|| 0)` with `map_or_else(|| 0, |x| x + 1)`
 101
 102 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
 103   --> $DIR/methods.rs:151:13
 104    |
 105 LL |       let _ = opt.map(|x| {
 106    |  _____________^
 107 LL | |                         x + 1
 108 LL | |                     }
 109 LL | |               ).unwrap_or_else(|| 0);
 110    | |____________________________________^
 111
 112 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
 113   --> $DIR/methods.rs:155:13
 114    |
 115 LL |       let _ = opt.map(|x| x + 1)
 116    |  _____________^
 117 LL | |                .unwrap_or_else(||
 118 LL | |                     0
 119 LL | |                 );
 120    | |_________________^
 121
 122 error: called `map_or(None, f)` on an Option value. This can be done more directly by calling `and_then(f)` instead
 123   --> $DIR/methods.rs:164:13
 124    |
 125 LL |     let _ = opt.map_or(None, |x| Some(x + 1));
 126    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try using and_then instead: `opt.and_then(|x| Some(x + 1))`
 127    |
 128    = note: `-D clippy::option-map-or-none` implied by `-D warnings`
 129
 130 error: called `map_or(None, f)` on an Option value. This can be done more directly by calling `and_then(f)` instead
 131   --> $DIR/methods.rs:166:13
 132    |
 133 LL |       let _ = opt.map_or(None, |x| {
 134    |  _____________^
 135 LL | |                         Some(x + 1)
 136 LL | |                        }
 137 LL | |                 );
 138    | |_________________^
 139 help: try using and_then instead
 140    |
 141 LL |     let _ = opt.and_then(|x| {
 142 LL |                         Some(x + 1)
 143 LL |                        });
 144    |
 145
 146 error: called `map(f).unwrap_or_else(g)` on a Result value. This can be done more directly by calling `ok().map_or_else(g, f)` instead
 147   --> $DIR/methods.rs:179:13
 148    |
 149 LL |       let _ = res.map(|x| x + 1)
 150    |  _____________^
 151 LL | |
 152 LL | |                .unwrap_or_else(|e| 0); // should lint even though this call is on a separate line
 153    | |_____________________________________^
 154    |
 155    = note: `-D clippy::result-map-unwrap-or-else` implied by `-D warnings`
 156    = note: replace `map(|x| x + 1).unwrap_or_else(|e| 0)` with `ok().map_or_else(|e| 0, |x| x + 1)`
 157
 158 error: called `map(f).unwrap_or_else(g)` on a Result value. This can be done more directly by calling `ok().map_or_else(g, f)` instead
 159   --> $DIR/methods.rs:183:13
 160    |
 161 LL |       let _ = res.map(|x| {
 162    |  _____________^
 163 LL | |                         x + 1
 164 LL | |                     }
 165 LL | |               ).unwrap_or_else(|e| 0);
 166    | |_____________________________________^
 167
 168 error: called `map(f).unwrap_or_else(g)` on a Result value. This can be done more directly by calling `ok().map_or_else(g, f)` instead
 169   --> $DIR/methods.rs:187:13
 170    |
 171 LL |       let _ = res.map(|x| x + 1)
 172    |  _____________^
 173 LL | |                .unwrap_or_else(|e|
 174 LL | |                     0
 175 LL | |                 );
 176    | |_________________^
 177
 178 error: called `filter(p).next()` on an `Iterator`. This is more succinctly expressed by calling `.find(p)` instead.
 179   --> $DIR/methods.rs:250:13
 180    |
 181 LL |     let _ = v.iter().filter(|&x| *x < 0).next();
 182    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 183    |
 184    = note: `-D clippy::filter-next` implied by `-D warnings`
 185    = note: replace `filter(|&x| *x < 0).next()` with `find(|&x| *x < 0)`
 186
 187 error: called `filter(p).next()` on an `Iterator`. This is more succinctly expressed by calling `.find(p)` instead.
 188   --> $DIR/methods.rs:253:13
 189    |
 190 LL |       let _ = v.iter().filter(|&x| {
 191    |  _____________^
 192 LL | |                                 *x < 0
 193 LL | |                             }
 194 LL | |                    ).next();
 195    | |___________________________^
 196
 197 error: called `is_some()` after searching an `Iterator` with find. This is more succinctly expressed by calling `any()`.
 198   --> $DIR/methods.rs:268:13
 199    |
 200 LL |     let _ = v.iter().find(|&x| *x < 0).is_some();
 201    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 202    |
 203    = note: `-D clippy::search-is-some` implied by `-D warnings`
 204    = note: replace `find(|&x| *x < 0).is_some()` with `any(|&x| *x < 0)`
 205
 206 error: called `is_some()` after searching an `Iterator` with find. This is more succinctly expressed by calling `any()`.
 207   --> $DIR/methods.rs:271:13
 208    |
 209 LL |       let _ = v.iter().find(|&x| {
 210    |  _____________^
 211 LL | |                               *x < 0
 212 LL | |                           }
 213 LL | |                    ).is_some();
 214    | |______________________________^
 215
 216 error: called `is_some()` after searching an `Iterator` with position. This is more succinctly expressed by calling `any()`.
 217   --> $DIR/methods.rs:277:13
 218    |
 219 LL |     let _ = v.iter().position(|&x| x < 0).is_some();
 220    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 221    |
 222    = note: replace `position(|&x| x < 0).is_some()` with `any(|&x| x < 0)`
 223
 224 error: called `is_some()` after searching an `Iterator` with position. This is more succinctly expressed by calling `any()`.
 225   --> $DIR/methods.rs:280:13
 226    |
 227 LL |       let _ = v.iter().position(|&x| {
 228    |  _____________^
 229 LL | |                                   x < 0
 230 LL | |                               }
 231 LL | |                    ).is_some();
 232    | |______________________________^
 233
 234 error: called `is_some()` after searching an `Iterator` with rposition. This is more succinctly expressed by calling `any()`.
 235   --> $DIR/methods.rs:286:13
 236    |
 237 LL |     let _ = v.iter().rposition(|&x| x < 0).is_some();
 238    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 239    |
 240    = note: replace `rposition(|&x| x < 0).is_some()` with `any(|&x| x < 0)`
 241
 242 error: called `is_some()` after searching an `Iterator` with rposition. This is more succinctly expressed by calling `any()`.
 243   --> $DIR/methods.rs:289:13
 244    |
 245 LL |       let _ = v.iter().rposition(|&x| {
 246    |  _____________^
 247 LL | |                                    x < 0
 248 LL | |                                }
 249 LL | |                    ).is_some();
 250    | |______________________________^
 251
 252 error: use of `unwrap_or` followed by a function call
 253   --> $DIR/methods.rs:324:22
 254    |
 255 LL |     with_constructor.unwrap_or(make());
 256    |                      ^^^^^^^^^^^^^^^^^ help: try this: `unwrap_or_else(make)`
 257    |
 258    = note: `-D clippy::or-fun-call` implied by `-D warnings`
 259
 260 error: use of `unwrap_or` followed by a call to `new`
 261   --> $DIR/methods.rs:327:5
 262    |
 263 LL |     with_new.unwrap_or(Vec::new());
 264    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `with_new.unwrap_or_default()`
 265
 266 error: use of `unwrap_or` followed by a function call
 267   --> $DIR/methods.rs:330:21
 268    |
 269 LL |     with_const_args.unwrap_or(Vec::with_capacity(12));
 270    |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `unwrap_or_else(|| Vec::with_capacity(12))`
 271
 272 error: use of `unwrap_or` followed by a function call
 273   --> $DIR/methods.rs:333:14
 274    |
 275 LL |     with_err.unwrap_or(make());
 276    |              ^^^^^^^^^^^^^^^^^ help: try this: `unwrap_or_else(|_| make())`
 277
 278 error: use of `unwrap_or` followed by a function call
 279   --> $DIR/methods.rs:336:19
 280    |
 281 LL |     with_err_args.unwrap_or(Vec::with_capacity(12));
 282    |                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `unwrap_or_else(|_| Vec::with_capacity(12))`
 283
 284 error: use of `unwrap_or` followed by a call to `default`
 285   --> $DIR/methods.rs:339:5
 286    |
 287 LL |     with_default_trait.unwrap_or(Default::default());
 288    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `with_default_trait.unwrap_or_default()`
 289
 290 error: use of `unwrap_or` followed by a call to `default`
 291   --> $DIR/methods.rs:342:5
 292    |
 293 LL |     with_default_type.unwrap_or(u64::default());
 294    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `with_default_type.unwrap_or_default()`
 295
 296 error: use of `unwrap_or` followed by a function call
 297   --> $DIR/methods.rs:345:14
 298    |
 299 LL |     with_vec.unwrap_or(vec![]);
 300    |              ^^^^^^^^^^^^^^^^^ help: try this: `unwrap_or_else(|| vec![])`
 301
 302 error: use of `unwrap_or` followed by a function call
 303   --> $DIR/methods.rs:350:21
 304    |
 305 LL |     without_default.unwrap_or(Foo::new());
 306    |                     ^^^^^^^^^^^^^^^^^^^^^ help: try this: `unwrap_or_else(Foo::new)`
 307
 308 error: use of `or_insert` followed by a function call
 309   --> $DIR/methods.rs:353:19
 310    |
 311 LL |     map.entry(42).or_insert(String::new());
 312    |                   ^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `or_insert_with(String::new)`
 313
 314 error: use of `or_insert` followed by a function call
 315   --> $DIR/methods.rs:356:21
 316    |
 317 LL |     btree.entry(42).or_insert(String::new());
 318    |                     ^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `or_insert_with(String::new)`
 319
 320 error: use of `unwrap_or` followed by a function call
 321   --> $DIR/methods.rs:359:21
 322    |
 323 LL |     let _ = stringy.unwrap_or("".to_owned());
 324    |                     ^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `unwrap_or_else(|| "".to_owned())`
 325
 326 error: called `.iter().nth()` on a Vec. Calling `.get()` is both faster and more readable
 327   --> $DIR/methods.rs:370:23
 328    |
 329 LL |         let bad_vec = some_vec.iter().nth(3);
 330    |                       ^^^^^^^^^^^^^^^^^^^^^^
 331    |
 332    = note: `-D clippy::iter-nth` implied by `-D warnings`
 333
 334 error: called `.iter().nth()` on a slice. Calling `.get()` is both faster and more readable
 335   --> $DIR/methods.rs:371:26
 336    |
 337 LL |         let bad_slice = &some_vec[..].iter().nth(3);
 338    |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^
 339
 340 error: called `.iter().nth()` on a slice. Calling `.get()` is both faster and more readable
 341   --> $DIR/methods.rs:372:31
 342    |
 343 LL |         let bad_boxed_slice = boxed_slice.iter().nth(3);
 344    |                               ^^^^^^^^^^^^^^^^^^^^^^^^^
 345
 346 error: called `.iter().nth()` on a VecDeque. Calling `.get()` is both faster and more readable
 347   --> $DIR/methods.rs:373:29
 348    |
 349 LL |         let bad_vec_deque = some_vec_deque.iter().nth(3);
 350    |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 351
 352 error: called `.iter_mut().nth()` on a Vec. Calling `.get_mut()` is both faster and more readable
 353   --> $DIR/methods.rs:378:23
 354    |
 355 LL |         let bad_vec = some_vec.iter_mut().nth(3);
 356    |                       ^^^^^^^^^^^^^^^^^^^^^^^^^^
 357
 358 error: called `.iter_mut().nth()` on a slice. Calling `.get_mut()` is both faster and more readable
 359   --> $DIR/methods.rs:381:26
 360    |
 361 LL |         let bad_slice = &some_vec[..].iter_mut().nth(3);
 362    |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 363
 364 error: called `.iter_mut().nth()` on a VecDeque. Calling `.get_mut()` is both faster and more readable
 365   --> $DIR/methods.rs:384:29
 366    |
 367 LL |         let bad_vec_deque = some_vec_deque.iter_mut().nth(3);
 368    |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 369
 370 error: called `skip(x).next()` on an iterator. This is more succinctly expressed by calling `nth(x)`
 371   --> $DIR/methods.rs:396:13
 372    |
 373 LL |     let _ = some_vec.iter().skip(42).next();
 374    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 375    |
 376    = note: `-D clippy::iter-skip-next` implied by `-D warnings`
 377
 378 error: called `skip(x).next()` on an iterator. This is more succinctly expressed by calling `nth(x)`
 379   --> $DIR/methods.rs:397:13
 380    |
 381 LL |     let _ = some_vec.iter().cycle().skip(42).next();
 382    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 383
 384 error: called `skip(x).next()` on an iterator. This is more succinctly expressed by calling `nth(x)`
 385   --> $DIR/methods.rs:398:13
 386    |
 387 LL |     let _ = (1..10).skip(10).next();
 388    |             ^^^^^^^^^^^^^^^^^^^^^^^
 389
 390 error: called `skip(x).next()` on an iterator. This is more succinctly expressed by calling `nth(x)`
 391   --> $DIR/methods.rs:399:14
 392    |
 393 LL |     let _ = &some_vec[..].iter().skip(3).next();
 394    |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 395
 396 error: used unwrap() on an Option value. If you don't want to handle the None case gracefully, consider using expect() to provide a better panic message
 397   --> $DIR/methods.rs:408:13
 398    |
 399 LL |     let _ = opt.unwrap();
 400    |             ^^^^^^^^^^^^
 401    |
 402    = note: `-D clippy::option-unwrap-used` implied by `-D warnings`
 403
 404 error: aborting due to 50 previous errors
 405