[ovs-dev] [python 06/31] python: Take advantage of Python "x < y < z" syntax.

Ben Pfaff blp at nicira.com
Wed Aug 24 18:59:02 UTC 2011


On Wed, Aug 24, 2011 at 12:28:00AM -0700, Reid Price wrote:
> On Tue, Aug 23, 2011 at 2:05 PM, Ben Pfaff <blp at nicira.com> wrote:
> 
> > Suggested-by: Reid Price <reid at nicira.com>
> > ---
> >  lib/ovsdb-types.c       |    3 +--
> >  python/ovs/db/data.py   |    2 +-
> >  python/ovs/db/parser.py |    4 ++--
> >  python/ovs/db/types.py  |    8 +++-----
> >  4 files changed, 7 insertions(+), 10 deletions(-)
> >
> > diff --git a/lib/ovsdb-types.c b/lib/ovsdb-types.c
> > index b3452dd..959f087 100644
> > --- a/lib/ovsdb-types.c
> > +++ b/lib/ovsdb-types.c
> > @@ -1,4 +1,4 @@
> > -/* Copyright (c) 2009, 2010 Nicira Networks
> > +/* Copyright (c) 2009, 2010, 2011 Nicira Networks
> >  *
> >  * Licensed under the Apache License, Version 2.0 (the "License");
> >  * you may not use this file except in compliance with the License.
> > @@ -556,7 +556,6 @@ ovsdb_type_is_valid(const struct ovsdb_type *type)
> >             && ovsdb_base_type_is_valid(&type->key)
> >             && ovsdb_base_type_is_valid(&type->value)
> >             && type->n_min <= 1
> > -            && type->n_min <= type->n_max
> >
> 
> I assume this one is on purpose too

Yes, I realized that if n_min <= 1 and n_max >= 1 then n_min <= n_max
is always true.



More information about the dev mailing list