Suggest replacing .map(f).flatten() with .and_then(f) on Options

[scottmcm]

What it does

What does this lint do?
Checks for usage of _.map(_).flatten(),

This is like https://rust-lang.github.io/rust-clippy/master/index.html#map_flatten but for Options instead of Iterators

Categories (optional)

• Kind: clippy::style/complexity

Easier to follow what's happening, shorter

Drawbacks

None.

Example

Inspired by @steffahn's suggestion in Nadrieril/dhall-rust#213 (comment)

[giraffate]

Do you mean this case?

rust-clippy/tests/ui/map_flatten.stderr

Lines 33 to 39 in 4c8d248

	error: called `map(..).flatten()` on an `Option`
	--> $DIR/map_flatten.rs:25:39
	|
	LL | let _: Option<_> = (Some(Some(1))).map(|x| x).flatten();
	| ^^^^^^^^^^^^^^^^^^^^^ help: try using `and_then` instead: `.and_then(|x| x)`
	error: aborting due to 6 previous errors

[scottmcm]

Ah, so I did! The description in the docs made me think it's only for Iterator::map.

[steffahn]

Which means the description should be fixed, right?

[giraffate]

Yes, I think so. I will prepare a fix.

[scottmcm]

Thanks, @giraffate!

[jyn514]

Hmm, this doesn't seem to be working: https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=d9855ae1539cc222b88fa4baa7b46b92

    let _: Option<&str> = x.as_ref().map(|pb| pb.to_str()).flatten();

doesn't get a warning.

[steffahn]

@jyn514 It’s a pedantic lint; not enabled by default.

https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=1e75f19dfbb4f5b35b2f2ef4a9d8b200