Re: [PATCH] backlight: replace usage of found with dedicated list iterator variable

On Thu, Mar 24, 2022 at 08:06:08AM +0100, Jakob Koschel wrote:

To move the list iterator variable into the list_for_each_entry_*() macro in the future it should be avoided to use the list iterator variable after the loop body.

To *never* use the list iterator variable after the loop it was concluded to use a separate iterator variable instead of a found boolean [1].

This removes the need to use a found variable and simply checking if the variable was set, can determine if the break/goto was hit.

Link: https://lore.kernel.org/all/CAHk-=wgRr_D8CB-D9Kg-c=EHreAsk5SqXPwr9Y7k9sA6cWX... Signed-off-by: Jakob Koschel jakobkoschel@gmail.com

Reviewed-by: Daniel Thompson daniel.thompson@linaro.org

Daniel.