identity_conversion false positive with `?` for loops in 1.35

[sfackler]

This may be a duplicate of #4081, but I can't easily check since clippy isn't in the current nightly.

fn foo() -> Result<Vec<u8>, ()> {
    Err(())
}

fn bar() -> Result<(), ()> {
    for _ in foo()? {}

    Ok(())
}

warning: identical conversion
 --> src/main.rs:6:14
  |
6 |     for _ in foo()? {}
  |              ^^^^^^ help: consider removing `foo()?()`: `foo()?`
  |
  = note: #[warn(clippy::identity_conversion)] on by default
  = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#identity_conversion

[hoodie]

ditto: seeing lots of those when I iterare over Result<Vec<_>> with ?

[baumanj]

Seeing this in 1.34 as well

[flip1995]

Yes this is fixed in #4083 and will be available in the next nightly Clippy.

[Manishearth]

Should we beta backport?

[flip1995]

I think that would be the right thing to do.

[oli-obk]

@flip1995 that issue link (4083) is not pointing to a PR. I have been unable to find the PR that fixed it.

[flip1995]

Oh sorry, it was the #4082 PR, the test for this is in #4089

[oli-obk]

I just checked, all of that PR is in the current beta, so we're good