Re: [ovs-dev] [PATCH 3/3] extract-ofp-fields: Detect duplicate fields.

2015-05-20 Thread Joe Stringer
On 20 May 2015 at 14:33, YAMAMOTO Takashi wrote: >> -class_ = oxm_name_to_class(name) >> +prefix, class_ = oxm_name_to_class(name) >> if class_ is None: >> fatal("unknown OXM class for %s" % name) >> oxm_vendor, oxm_class = class_ >> >> +if prefix in match_types: >>

Re: [ovs-dev] [PATCH 3/3] extract-ofp-fields: Detect duplicate fields.

2015-05-20 Thread YAMAMOTO Takashi
> -class_ = oxm_name_to_class(name) > +prefix, class_ = oxm_name_to_class(name) > if class_ is None: > fatal("unknown OXM class for %s" % name) > oxm_vendor, oxm_class = class_ > > +if prefix in match_types: > +if oxm_type in match_types[prefix]: > +

[ovs-dev] [PATCH 3/3] extract-ofp-fields: Detect duplicate fields.

2015-05-20 Thread Joe Stringer
Figure out if a developer accidentally defines new NXM fields using an existing number, and warn them. Useful particularly if new fields are introduced upstream while rebasing an in-progress patchset. Signed-off-by: Joe Stringer --- build-aux/extract-ofp-fields | 17 +++-- 1 file cha