Ignore macros with ! operators in eq_op

[JohnTitor]

SpanlessEq::eq_expr doesn't ignore macros with ! operators and I'm not sure we should ignore there, so I ignore in eq_op (and op_ref).

Fixes #5077

changelog: Fix false positive in eq_op

[flip1995]

I think this issue not only exists for !macro!(..), but also for just macro!(..). So cfg!(feature = "a") == cfg!(feature = "b") still gets linted. Can you add a test for this case?

[JohnTitor]

I think this issue not only exists for !macro!(..), but also for just macro!(..). So cfg!(feature = "a") == cfg!(feature = "b") still gets linted.

Uhm, I cannot reproduce it: https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=b385ae2aeab61bd9ee50ca74204209ff
I may be wrong so could you provide an example via playground?

[flip1995]

I may be wrong so could you provide an example via playground?

Nope, seems this is already handled. All good then.

@bors r+

[bors]

📌 Commit a7d58ed has been approved by flip1995

[bors]

⌛ Testing commit a7d58ed with merge eff3bc5...

[bors]

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing eff3bc5 to master...