useless destructive match

What it does

pub fn a(x: Option<i32>) -> Option<i32> {
    if let Some(x) = x {
        return Some(x);
    }
    // could just be be:
    if x.is_some() {
       return x
    }

     // could probably lint Result<> as well, bot sure about any enum container in general though
    todo!()
}

pub fn b(x: Option<i32>) -> Option<i32> {
    return match x {
        Some(x) => Some(x),
        _ => todo!(),
    };
    // could be:, but if-let is probably more idiomatic 
    return match x {
        None => todo!(),
        x => x, 
    };

    }
```

Start by comparing the Rust examples for the proposed if let and match patterns and determine the intended lint scope, including whether Result or other enum containers are covered. No source file or test is named; done would require an agreed scope and corresponding lint behavior and tests.