if_same_then_else should ignore empty bodies

Summary

Example:

            if x.ident == y {
            } else if first && (*y == "core" || *y == "alloc") && x.ident == "std" {
            } else {
                return false;
            }

Yes, this is just a chained if expression. There are different ways to write it; as a single expression with/without brackets, via let bindings (though it's less clear if these take advantage of lazy evaluation), even via a match expression.

This is all about style (which, in my opinion, Clippy does not deserve a say in) and nothing about efficiency or redundancy.

Lint Name

if_same_then_else

Reproducer

No response

Version

No response

Additional Labels

No response

Start at the if_same_then_else lint implementation and its existing tests in rust-clippy. Trace how chained if branches are classified, then add coverage for branches with empty bodies. Done means the lint ignores empty branches in the reported pattern while preserving its existing behavior for non-empty branches.