Messages in this thread | | | From | "Sa, Nuno" <> | Subject | RE: [PATCH 3/3] iio: sysfs-trigger: replace usage of found with dedicated list iterator variable | Date | Fri, 1 Apr 2022 12:41:58 +0000 |
| |
> -----Original Message----- > From: Jakob Koschel <jakobkoschel@gmail.com> > Sent: Friday, April 1, 2022 1:07 AM > To: Jonathan Cameron <jic23@kernel.org> > Cc: Lars-Peter Clausen <lars@metafoo.de>; Dan Carpenter > <dan.carpenter@oracle.com>; Jakob Koschel > <jakobkoschel@gmail.com>; linux-iio@vger.kernel.org; linux- > kernel@vger.kernel.org; Mike Rapoport <rppt@kernel.org>; Brian > Johannesmeyer <bjohannesmeyer@gmail.com>; Cristiano Giuffrida > <c.giuffrida@vu.nl>; Bos, H.J. <h.j.bos@vu.nl> > Subject: [PATCH 3/3] iio: sysfs-trigger: replace usage of found with > dedicated list iterator variable > > [External] > > 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://urldefense.com/v3/__https://lore.kernel.org/all/CAHk- > =wgRr_D8CB-D9Kg- > c=EHreAsk5SqXPwr9Y7k9sA6cWXJ6w@mail.gmail.com/__;!!A3Ni8CS0y > 2Y!q8mEciLiCAWjMGEwW9jHDD2DoeaKzrMwtUcRCNAZ0gW9DPBVtJk > xR5FMPPCJNw$ [1] > Signed-off-by: Jakob Koschel <jakobkoschel@gmail.com> > ---
Reviewed-by: Nuno Sá <nuno.sa@analog.com>
| |