From: Eric Sunshine Date: Wed, 31 Oct 2018 17:28:34 GMT Subject: Re: [PATCH v2] sequencer: break out of loop explicitly Message-ID: In-Reply-To: On Wed, Oct 31, 2018 at 10:54 AM Johannes Schindelin wrote: > On Tue, 30 Oct 2018, Martin Ågren wrote: > > Rewrite the loop to a more idiomatic variant which doesn't muck with > > `len` in the loop body. That should help compilers and human readers > > figure out what is going on here. But do note that we need to update > > `len` since it is not only used just after this loop (where we could > > have used `i` directly), but also later in this function. > > > > Signed-off-by: Martin Ågren > > --- > > ACK. Thanks for cleaning up after me, Looks good to me, as well. Thanks for working on it.