Fix empty_line_after_outer_attribute false positive

[smoelius]

This PR fixes a false positive in empty_line_after_outer_attribute.

Here is a minimal example that trigger the FP:

#[derive(clap::Parser)]
#[clap(after_help = "This ia a help message.

You're welcome.
")]
pub struct Args;

changelog: PF: [empty_line_after_outer_attribute]: No longer lints empty lines in inner string values.

[rust-highfive]

r? @Manishearth

(rust-highfive has picked a reviewer for you, use r? to override)

[Manishearth]

@bors r+

thanks!

[bors]

📌 Commit 9ee211a has been approved by Manishearth

[bors]

⌛ Testing commit 9ee211a with merge 7438662...

[bors]

💔 Test failed - checks-action_test

[xFrednet]

The changelog was missing. I hope my addition is a correct description of the fix 🙃

[xFrednet]

@bors retry

[bors]

⌛ Testing commit 9ee211a with merge 1dd5547...

[bors]

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Manishearth
Pushing 1dd5547 to master...

[flip1995]

Uh, this added clap as another UI test dep to the Cargo.toml. I think this could've easily been avoided by writing a small custom proc_macro, that would've also reproduced this.

[flip1995]

I fixed this during the sync to rustc with commit rust-lang/rust@c22d4e6

[smoelius]

Sorry, I wasn't aware of proc_macro_attr.rs.

[flip1995]

All good, you live and learn: In UI tests you can depend on files/crates in the auxiliary directory. Those can even be proc_macro crates. This is helpful if you want to test lints that check interactions with external crates, like is_external_macro or like in this case proc_macros.