src/tools/clippy/tests/ui/or_fun_call.fixed

#![warn(clippy::or_fun_call)]
#![allow(
    clippy::borrow_as_ptr,
    clippy::uninlined_format_args,
    clippy::unnecessary_wraps,
    clippy::unnecessary_literal_unwrap,
    clippy::unnecessary_result_map_or_else,
    clippy::unnecessary_option_map_or_else,
    clippy::useless_vec
)]

use std::collections::{BTreeMap, HashMap};
use std::time::Duration;

/// Checks implementation of the `OR_FUN_CALL` lint.
fn or_fun_call() {
    struct Foo;

    impl Foo {
        fn new() -> Foo {
            Foo
        }
    }

    struct FakeDefault;
    impl FakeDefault {
        fn default() -> Self {
            FakeDefault
        }
    }

    impl Default for FakeDefault {
        fn default() -> Self {
            FakeDefault
        }
    }

    enum Enum {
        A(i32),
    }

    fn make<T>() -> T {
        unimplemented!();
    }

    let with_enum = Some(Enum::A(1));
    with_enum.unwrap_or(Enum::A(5));

    let with_const_fn = Some(Duration::from_secs(1));
    with_const_fn.unwrap_or(Duration::from_secs(5));

    let with_constructor = Some(vec![1]);
    with_constructor.unwrap_or_else(make);
    //~^ or_fun_call

    let with_new: Option<Vec<i32>> = Some(vec![1]);
    with_new.unwrap_or_default();
    //~^ unwrap_or_default

    let with_const_args = Some(vec![1]);
    with_const_args.unwrap_or_else(|| Vec::with_capacity(12));
    //~^ or_fun_call

    let with_err: Result<_, ()> = Ok(vec![1]);
    with_err.unwrap_or_else(|_| make());
    //~^ or_fun_call

    let with_err_args: Result<_, ()> = Ok(vec![1]);
    with_err_args.unwrap_or_else(|_| Vec::with_capacity(12));
    //~^ or_fun_call

    let with_default_trait = Some(1);
    with_default_trait.unwrap_or_default();
    //~^ unwrap_or_default

    let with_default_type = Some(1);
    with_default_type.unwrap_or_default();
    //~^ unwrap_or_default

    let with_default_literal = Some(1);
    with_default_literal.unwrap_or(0);
    // Do not lint because `.unwrap_or_default()` wouldn't be simpler

    let with_default_literal = Some(1.0);
    with_default_literal.unwrap_or(0.0);
    // Do not lint because `.unwrap_or_default()` wouldn't be simpler

    let with_default_literal = Some("foo");
    with_default_literal.unwrap_or("");
    // Do not lint because `.unwrap_or_default()` wouldn't be simpler

    let with_default_vec_macro = Some(vec![1, 2, 3]);
    with_default_vec_macro.unwrap_or(vec![]);
    // Do not lint because `.unwrap_or_default()` wouldn't be simpler

    let self_default = None::<FakeDefault>;
    self_default.unwrap_or_else(<FakeDefault>::default);
    //~^ or_fun_call

    let real_default = None::<FakeDefault>;
    real_default.unwrap_or_default();
    //~^ unwrap_or_default

    let with_vec: Option<Vec<i32>> = Some(vec![1]);
    with_vec.unwrap_or_default();
    //~^ unwrap_or_default

    let without_default = Some(Foo);
    without_default.unwrap_or_else(Foo::new);
    //~^ or_fun_call

    let mut map = HashMap::<u64, String>::new();
    map.entry(42).or_default();
    //~^ unwrap_or_default

    let mut map_vec = HashMap::<u64, Vec<i32>>::new();
    map_vec.entry(42).or_default();
    //~^ unwrap_or_default

    let mut btree = BTreeMap::<u64, String>::new();
    btree.entry(42).or_default();
    //~^ unwrap_or_default

    let mut btree_vec = BTreeMap::<u64, Vec<i32>>::new();
    btree_vec.entry(42).or_default();
    //~^ unwrap_or_default

    let stringy = Some(String::new());
    let _ = stringy.unwrap_or_default();
    //~^ unwrap_or_default

    let opt = Some(1);
    let hello = "Hello";
    let _ = opt.ok_or_else(|| format!("{} world.", hello));
    //~^ or_fun_call

    // index
    let map = HashMap::<u64, u64>::new();
    let _ = Some(1).unwrap_or_else(|| map[&1]);
    //~^ or_fun_call
    let map = BTreeMap::<u64, u64>::new();
    let _ = Some(1).unwrap_or_else(|| map[&1]);
    //~^ or_fun_call
    // don't lint index vec
    let vec = vec![1];
    let _ = Some(1).unwrap_or(vec[1]);
}

struct Foo(u8);
struct Bar(String, Duration);
#[rustfmt::skip]
fn test_or_with_ctors() {
    let opt = Some(1);
    let opt_opt = Some(Some(1));
    // we also test for const promotion, this makes sure we don't hit that
    let two = 2;

    let _ = opt_opt.unwrap_or(Some(2));
    let _ = opt_opt.unwrap_or(Some(two));
    let _ = opt.ok_or(Some(2));
    let _ = opt.ok_or(Some(two));
    let _ = opt.ok_or(Foo(2));
    let _ = opt.ok_or(Foo(two));
    let _ = opt.or(Some(2));
    let _ = opt.or(Some(two));

    let _ = Some("a".to_string()).or_else(|| Some("b".to_string()));
    //~^ or_fun_call

    let b = "b".to_string();
    let _ = Some(Bar("a".to_string(), Duration::from_secs(1)))
        .or(Some(Bar(b, Duration::from_secs(2))));

    let vec = vec!["foo"];
    let _ = opt.ok_or(vec.len());

    let array = ["foo"];
    let _ = opt.ok_or(array.len());

    let slice = &["foo"][..];
    let _ = opt.ok_or(slice.len());

    let string = "foo";
    let _ = opt.ok_or(string.len());
}

// Issue 4514 - early return
fn f() -> Option<()> {
    let a = Some(1);
    let b = 1i32;

    let _ = a.unwrap_or(b.checked_mul(3)?.min(240));

    Some(())
}

mod issue6675 {
    unsafe fn ptr_to_ref<'a, T>(p: *const T) -> &'a T {
        unsafe {
            #[allow(unused)]
            let x = vec![0; 1000]; // future-proofing, make this function expensive.
            &*p
        }
    }

    unsafe fn foo() {
        unsafe {
            let s = "test".to_owned();
            let s = &s as *const _;
            None.unwrap_or_else(|| ptr_to_ref(s));
            //~^ or_fun_call
        }
    }

    fn bar() {
        let s = "test".to_owned();
        let s = &s as *const _;
        None.unwrap_or_else(|| unsafe { ptr_to_ref(s) });
        //~^ or_fun_call
        #[rustfmt::skip]
        None.unwrap_or_else(|| unsafe { ptr_to_ref(s) });
        //~^ or_fun_call
    }
}

mod issue8239 {
    fn more_than_max_suggestion_highest_lines_0() {
        let frames = Vec::new();
        frames
            .iter()
            .map(|f: &String| f.to_lowercase())
            .reduce(|mut acc, f| {
                acc.push_str(&f);
                acc
            })
            .unwrap_or(String::new());
    }

    fn more_to_max_suggestion_highest_lines_1() {
        let frames = Vec::new();
        let iter = frames.iter();
        iter.map(|f: &String| f.to_lowercase())
            .reduce(|mut acc, f| {
                let _ = "";
                let _ = "";
                acc.push_str(&f);
                acc
            })
            .unwrap_or(String::new());
    }

    fn equal_to_max_suggestion_highest_lines() {
        let frames = Vec::new();
        let iter = frames.iter();
        iter.map(|f: &String| f.to_lowercase())
            .reduce(|mut acc, f| {
                let _ = "";
                acc.push_str(&f);
                acc
            })
            .unwrap_or(String::new());
    }

    fn less_than_max_suggestion_highest_lines() {
        let frames = Vec::new();
        let iter = frames.iter();
        let map = iter.map(|f: &String| f.to_lowercase());
        map.reduce(|mut acc, f| {
            acc.push_str(&f);
            acc
        })
        .unwrap_or(String::new());
    }
}

mod issue9608 {
    fn sig_drop() {
        enum X {
            X(std::fs::File),
            Y(u32),
        }

        let _ = None.unwrap_or(X::Y(0));
    }
}

mod issue8993 {
    fn g() -> i32 {
        3
    }

    fn f(n: i32) -> i32 {
        n
    }

    fn test_map_or() {
        let _ = Some(4).map_or_else(g, |v| v);
        //~^ or_fun_call
        let _ = Some(4).map_or_else(g, f);
        //~^ or_fun_call
        let _ = Some(4).map_or(0, f);
        let _ = Some(4).map_or_else(|| "asd".to_string().len() as i32, f);
        //~^ or_fun_call
    }
}

mod lazy {
    use super::*;

    fn foo() {
        struct Foo;

        impl Foo {
            fn new() -> Foo {
                Foo
            }
        }

        struct FakeDefault;
        impl FakeDefault {
            fn default() -> Self {
                FakeDefault
            }
        }

        impl Default for FakeDefault {
            fn default() -> Self {
                FakeDefault
            }
        }

        let with_new: Option<Vec<i32>> = Some(vec![1]);
        with_new.unwrap_or_default();
        //~^ unwrap_or_default

        let with_default_trait = Some(1);
        with_default_trait.unwrap_or_default();
        //~^ unwrap_or_default

        let with_default_type = Some(1);
        with_default_type.unwrap_or_default();
        //~^ unwrap_or_default

        let real_default = None::<FakeDefault>;
        real_default.unwrap_or_default();
        //~^ unwrap_or_default

        let mut map = HashMap::<u64, String>::new();
        map.entry(42).or_default();
        //~^ unwrap_or_default

        let mut btree = BTreeMap::<u64, String>::new();
        btree.entry(42).or_default();
        //~^ unwrap_or_default

        let stringy = Some(String::new());
        let _ = stringy.unwrap_or_default();
        //~^ unwrap_or_default

        // negative tests
        let self_default = None::<FakeDefault>;
        self_default.unwrap_or_else(<FakeDefault>::default);

        let without_default = Some(Foo);
        without_default.unwrap_or_else(Foo::new);
    }
}

fn host_effect() {
    // #12877 - make sure we don't ICE in type_certainty
    use std::ops::Add;

    Add::<i32>::add(1, 1).add(i32::MIN);
}

mod issue_10228 {
    struct Entry;

    impl Entry {
        fn or_insert(self, _default: i32) {}
        fn or_default(self) {
            // Don't lint, suggested code is an infinite recursion
            self.or_insert(Default::default())
        }
    }
}

// issue #12973
fn fn_call_in_nested_expr() {
    struct Foo {
        val: String,
    }

    fn f() -> i32 {
        1
    }
    let opt: Option<i32> = Some(1);

    let _ = opt.unwrap_or_else(f); // suggest `.unwrap_or_else(f)`
    //
    //~^^ or_fun_call
    //

    let _ = opt.unwrap_or_else(|| f() + 1); // suggest `.unwrap_or_else(|| f() + 1)`
    //
    //~^^ or_fun_call
    //

    let _ = opt.unwrap_or_else(|| {
        //~^ or_fun_call
        let x = f();
        x + 1
    });

    let _ = opt.map_or_else(|| f() + 1, |v| v); // suggest `.map_or_else(|| f() + 1, |v| v)`
    //
    //~^^ or_fun_call
    //

    let _ = opt.unwrap_or_default();
    //~^ unwrap_or_default

    let opt_foo = Some(Foo {
        val: String::from("123"),
    });

    let _ = opt_foo.unwrap_or_else(|| Foo { val: String::default() });
    //~^ or_fun_call
}

mod result_map_or {
    fn g() -> i32 {
        3
    }

    fn f(n: i32) -> i32 {
        n
    }

    fn test_map_or() {
        let x: Result<i32, ()> = Ok(4);
        let _ = x.map_or_else(|_| g(), |v| v);
        //~^ or_fun_call
        let _ = x.map_or_else(|_| g(), f);
        //~^ or_fun_call
        let _ = x.map_or(0, f);
        let _ = x.map_or_else(|_| "asd".to_string().len() as i32, f);
        //~^ or_fun_call
    }
}

fn test_option_get_or_insert() {
    // assume that this is slow call
    fn g() -> u8 {
        99
    }
    let mut x = Some(42_u8);
    let _ = x.get_or_insert_with(g);
    //~^ or_fun_call
}

fn test_option_and() {
    // assume that this is slow call
    fn g() -> Option<u8> {
        Some(99)
    }
    let mut x = Some(42_u8);
    let _ = x.and_then(|_| g());
    //~^ or_fun_call
}

fn test_result_and() {
    // assume that this is slow call
    fn g() -> Result<u8, ()> {
        Ok(99)
    }
    let mut x: Result<u8, ()> = Ok(42);
    let _ = x.and_then(|_| g());
    //~^ or_fun_call
}

#[clippy::msrv = "1.15"]
fn below_msrv(opt: Option<i32>, res: Result<i32, ()>) {
    let _ = opt.unwrap_or_default();
    //~^ unwrap_or_default
    let _ = res.unwrap_or_else(|_| Default::default());
    //~^ or_fun_call
}

#[clippy::msrv = "1.16"]
fn above_msrv(opt: Option<i32>, res: Result<i32, ()>) {
    let _ = opt.unwrap_or_default();
    //~^ unwrap_or_default
    let _ = res.unwrap_or_default();
    //~^ unwrap_or_default
}
fn main() {}