needless_range_loop: improve documentation

[tylerjw]

fixes #6929

changelog: [needless_range_loop]: improve documentation

[rustbot]

r? @Manishearth

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

[ChrisJefferson]

I think there are further improvements to be made here, but this is an improvement over what was there before.

[tylerjw]

What would you like it to also do?

[ChrisJefferson]

Sorry, I should have been clearer.

There is also #6930 , which discusses how this needless_range_loop can change behaviour -- however I think that should be dealt with separately, if for no other reason than it's not clear (to me) if the fix should be making this clear in the docs, or changing the suggestion.

[tylerjw]

Thank you for clarifying. I read that other issue and I'm unsure what change to make for it, that is why I only linked the issue about clarifying the docs.

[Manishearth]

@bors r+

thanks!

[bors]

📌 Commit 5ed191d has been approved by Manishearth

It is now in the queue for this repository.

[bors]

⌛ Testing commit 5ed191d with merge d92070a...

[bors]

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