Make for_each_expr visit closures by default, rename the old version for_each_expr_without_closures

[Alexendoo]

A lot of the time for_each_expr is picked when closures should be visited so I think it makes sense for this to be the default with the alternative available for when you don't need to visit them.

The first commit renames for_each_expr to for_each_expr_without_closures and for_each_expr_with_closures to for_each_expr

The second commit switches a few uses that I caught over to include closures to fix a few bugs

changelog: none

[rustbot]

r? @dswij

rustbot has assigned @dswij.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer