find_all_ret_expressions: change bool to Result

[GrigorenkoPV]

I think the semantics are clearer this way

---

changelog: none

[rustbot]

r? @llogiq

rustbot has assigned @llogiq.
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

Why was this reviewer chosen?

The reviewer was selected based on:

• Owners of files modified in this PR: 7 candidates
• 7 candidates expanded to 7 candidates
• Random selection from Jarcho, dswij, llogiq, samueltardieu

[llogiq]

I don't see any benefit from this. It adds four lines of code, too. So unless you can give me a good reason not to, I would favor closing this PR.