2023-04-20 09:37:15 -05:00
|
|
|
//@aux-build:option_helpers.rs
|
2022-10-21 16:35:39 -05:00
|
|
|
|
2020-05-03 12:58:27 -05:00
|
|
|
#![warn(clippy::map_unwrap_or)]
|
2022-10-21 16:35:39 -05:00
|
|
|
#![allow(clippy::uninlined_format_args, clippy::unnecessary_lazy_evaluations)]
|
2019-10-05 07:23:52 -05:00
|
|
|
|
|
|
|
#[macro_use]
|
|
|
|
extern crate option_helpers;
|
|
|
|
|
|
|
|
use std::collections::HashMap;
|
|
|
|
|
|
|
|
#[rustfmt::skip]
|
|
|
|
fn option_methods() {
|
|
|
|
let opt = Some(1);
|
|
|
|
|
2020-05-03 05:28:40 -05:00
|
|
|
// Check for `option.map(_).unwrap_or(_)` use.
|
2020-10-26 05:02:01 -05:00
|
|
|
// Single line case.
|
|
|
|
let _ = opt.map(|x| x + 1)
|
|
|
|
// Should lint even though this call is on a separate line.
|
|
|
|
.unwrap_or(0);
|
2019-10-05 07:23:52 -05:00
|
|
|
// Multi-line cases.
|
|
|
|
let _ = opt.map(|x| {
|
|
|
|
x + 1
|
|
|
|
}
|
|
|
|
).unwrap_or(0);
|
|
|
|
let _ = opt.map(|x| x + 1)
|
|
|
|
.unwrap_or({
|
|
|
|
0
|
|
|
|
});
|
|
|
|
// Single line `map(f).unwrap_or(None)` case.
|
|
|
|
let _ = opt.map(|x| Some(x + 1)).unwrap_or(None);
|
|
|
|
// Multi-line `map(f).unwrap_or(None)` cases.
|
|
|
|
let _ = opt.map(|x| {
|
|
|
|
Some(x + 1)
|
|
|
|
}
|
|
|
|
).unwrap_or(None);
|
|
|
|
let _ = opt
|
|
|
|
.map(|x| Some(x + 1))
|
|
|
|
.unwrap_or(None);
|
|
|
|
// macro case
|
|
|
|
let _ = opt_map!(opt, |x| x + 1).unwrap_or(0); // should not lint
|
|
|
|
|
|
|
|
// Should not lint if not copyable
|
|
|
|
let id: String = "identifier".to_string();
|
|
|
|
let _ = Some("prefix").map(|p| format!("{}.{}", p, id)).unwrap_or(id);
|
|
|
|
// ...but DO lint if the `unwrap_or` argument is not used in the `map`
|
|
|
|
let id: String = "identifier".to_string();
|
|
|
|
let _ = Some("prefix").map(|p| format!("{}.", p)).unwrap_or(id);
|
|
|
|
|
2020-05-03 05:28:40 -05:00
|
|
|
// Check for `option.map(_).unwrap_or_else(_)` use.
|
2019-10-05 07:23:52 -05:00
|
|
|
// Multi-line cases.
|
|
|
|
let _ = opt.map(|x| {
|
|
|
|
x + 1
|
|
|
|
}
|
|
|
|
).unwrap_or_else(|| 0);
|
|
|
|
let _ = opt.map(|x| x + 1)
|
|
|
|
.unwrap_or_else(||
|
|
|
|
0
|
|
|
|
);
|
2020-05-03 05:28:40 -05:00
|
|
|
}
|
|
|
|
|
2020-10-26 05:02:01 -05:00
|
|
|
#[rustfmt::skip]
|
|
|
|
fn result_methods() {
|
|
|
|
let res: Result<i32, ()> = Ok(1);
|
|
|
|
|
|
|
|
// Check for `result.map(_).unwrap_or_else(_)` use.
|
|
|
|
// multi line cases
|
|
|
|
let _ = res.map(|x| {
|
|
|
|
x + 1
|
|
|
|
}
|
|
|
|
).unwrap_or_else(|_e| 0);
|
|
|
|
let _ = res.map(|x| x + 1)
|
|
|
|
.unwrap_or_else(|_e| {
|
|
|
|
0
|
|
|
|
});
|
|
|
|
// macro case
|
|
|
|
let _ = opt_map!(res, |x| x + 1).unwrap_or_else(|_e| 0); // should not lint
|
|
|
|
}
|
|
|
|
|
2019-10-05 07:23:52 -05:00
|
|
|
fn main() {
|
|
|
|
option_methods();
|
2020-10-26 05:02:01 -05:00
|
|
|
result_methods();
|
2019-10-05 07:23:52 -05:00
|
|
|
}
|
2022-10-21 16:35:39 -05:00
|
|
|
|
2022-11-19 06:50:02 -06:00
|
|
|
#[clippy::msrv = "1.40"]
|
2022-10-21 16:35:39 -05:00
|
|
|
fn msrv_1_40() {
|
|
|
|
let res: Result<i32, ()> = Ok(1);
|
|
|
|
|
|
|
|
let _ = res.map(|x| x + 1).unwrap_or_else(|_e| 0);
|
|
|
|
}
|
|
|
|
|
2022-11-19 06:50:02 -06:00
|
|
|
#[clippy::msrv = "1.41"]
|
2022-10-21 16:35:39 -05:00
|
|
|
fn msrv_1_41() {
|
|
|
|
let res: Result<i32, ()> = Ok(1);
|
|
|
|
|
|
|
|
let _ = res.map(|x| x + 1).unwrap_or_else(|_e| 0);
|
|
|
|
}
|