Fix FP of single-element-loop

[tamaroning]

closes #10018

---

changelog: [single_element_loop]: No longer lints, if the loop contains a break or continue
#10162

[rustbot]

r? @flip1995

(rustbot has picked a reviewer for you, use r? to override)

[xFrednet]

r? @xFrednet :)

I wanted to look at the changes anyways to update the changelog entry, and @flip1995 won't mind one review less xD

[xFrednet]

LGTM, I only have one small suggestion, then we can merge this.

Side note: I wonder if it would make more sense, to simply change the applicability and point out, that the break and continuestatement should be refactored. However, that seems like a lot of effort for little benefit. Some users might even use this pattern to structure their control flow (Even if that might not be ideal). Therefore, I think we should keep this implementation :)

[tamaroning]

@xFrednet thank you for your review. I've finished fixing code.

[xFrednet]

You're welcome, thank you for the swift response. This looks good to me.

@bors r+

[bors]

📌 Commit ce56cf7 has been approved by xFrednet

It is now in the queue for this repository.

[bors]

⌛ Testing commit ce56cf7 with merge cf1d3d0...

[bors]

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: xFrednet
Pushing cf1d3d0 to master...