[ovs-discuss] no-op range checks

Ben Pfaff blp at nicira.com
Fri Apr 15 16:42:32 UTC 2011


On Fri, Apr 15, 2011 at 11:48:46AM +0900, YAMAMOTO Takashi wrote:
> > On Wed, Apr 13, 2011 at 12:37:28PM +0900, YAMAMOTO Takashi wrote:
> >> commit bb5bc6c042a22103612205483558f472f8929356
> >> Author: Ben Pfaff <blp at nicira.com>
> >> Date:   Mon Mar 21 14:30:33 2011 -0700
> >> 
> >>     lacp: Encapsulate configuration into new structs.
> >> 
> >>     This makes it easier to pass configuration between modules.
> >> 
> >> this commit made some range checks useless.
> > 
> > Can you post all of the warnings?  I don't get any of these with my
> > compiler.
> 
> actually there seems to be only a single instance related to the change.

Thanks for the full set of warnings.  I posted a series of three
patches that should fix them.  Would you mind build-testing them to
make sure that it does successfully suppress the warnings for you?

Thanks,

Ben.



More information about the discuss mailing list