On Tue, Mar 07, 2017 at 08:03:16PM +0100, Greg Kroah-Hartman wrote:
> On Tue, Mar 07, 2017 at 01:29:35PM +1100, Tobin C. Harding wrote:
> > Loop invariant is inside the loop so code checks invariant on each
> > iteration of the loop. Invariant can be moved outside of the loop so
> > it is only checked once.
> > 
> > Move loop invariant outside of for loop.
> 
> But does it really matter?  Does this fix an issue?  Make something
> faster in a _measurable_ way?  Did you test this on the hardware?

Good points, patch is untested and does not fix a particular
issue. Lesson learned.

thanks,
Tobin.
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to