Clippy lints for tracing::instrument'd code gets an incorrect "unneeded return" suggestion as of 0.1.29

[grahamc]

#3108 fixed some really great issues with macro'd functions. However, I noticed a function with an unneeded return statement incorrectly kept the ; in the suggestion:

error: unneeded `return` statement
  --> src/keychain.rs:97:5
   |
97 |     return Ok(());
   |     ^^^^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
   = note: `-D clippy::needless-return` implied by `-D warnings`
   = help: to override `-D warnings` add `#[allow(clippy::needless_return)]`
help: remove `return`
   |
79 | #[tracing::instrument(skip(password))]
...
96 |
97 ~     Ok(());
   |

That suggestion for line 97 needs to also remove the ;.

Originally posted by @grahamc in #3108 (comment)

[jplatte]

Can reproduce with just

#[instrument]
fn foo() {
    return;
}

An unfortunate thing about this is that it's a clippy lint, so I can't make a trybuild test for it 🤔