On Thu, Jun 30, 2011 at 09:36:57AM -0700, Ben Pfaff wrote:
> On Thu, Jun 30, 2011 at 08:34:15PM +0900, Simon Horman wrote:
> > The existing dual-loop setup is unnecessary
> > as the outer loop only skips to the first non-zero value
> > and then exits once the inner loop completes.
> > Zero values in the inner loop have no affect on its logic.
> > 
> > Signed-off-by: Simon Horman <[email protected]>
> > 
> > ---
> > v2 Initialise bucket
> > 
> > Compile tested only
> 
> Fair enough.
> 
> I touched it up to suit my notion of style and pushed it as follows.

Thanks, I was a little unsure about what to do about the
variable declarations and initialisations.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to