[igt-dev] [PATCH i-g-t] tests/i915/gen_mmap: fix no-op loops
Ser, Simon
simon.ser at intel.com
Tue May 7 15:10:46 UTC 2019
On Tue, 2019-05-07 at 15:31 +0100, Chris Wilson wrote:
> Quoting Simon Ser (2019-05-07 13:44:41)
> > The loop condition is never satisfied, since after filling the
> > array i > 0. For
> > this reason the loop is always a no-op.
> >
> > Use a more conventional loop instead.
>
> Or a classic while loop? Bad me for not reading the check was
> back-to-front.
You mean while(i--)?
I generally prioritize code clarity in these situations, hence the for
loop. Also not a fan of decrementing operators returning a value.
More information about the igt-dev
mailing list