Thanks, I pushed this to master.
On Tue, Jul 24, 2012 at 03:45:58PM -0700, Ethan Jackson wrote: > Looks good to me. > > Ethan > > On Tue, Jul 24, 2012 at 3:43 PM, Ben Pfaff <b...@nicira.com> wrote: > > This fixes the following warning from "sparse": > > > > meta-flow.c:830:31: warning: incorrect type in return expression (different > > base types) > > meta-flow.c:830:31: expected bool > > meta-flow.c:830:31: got restricted __be16 > > > > Signed-off-by: Ben Pfaff <b...@nicira.com> > > --- > > lib/meta-flow.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/lib/meta-flow.c b/lib/meta-flow.c > > index c50cfc5..3713ca4 100644 > > --- a/lib/meta-flow.c > > +++ b/lib/meta-flow.c > > @@ -827,7 +827,7 @@ mf_are_prereqs_ok(const struct mf_field *mf, const > > struct flow *flow) > > case MFP_IPV6: > > return flow->dl_type == htons(ETH_TYPE_IPV6); > > case MFP_VLAN_VID: > > - return flow->vlan_tci & htons(VLAN_CFI); > > + return (flow->vlan_tci & htons(VLAN_CFI)) != 0; > > case MFP_IP_ANY: > > return is_ip_any(flow); > > > > -- > > 1.7.2.5 > > > > _______________________________________________ > > dev mailing list > > dev@openvswitch.org > > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
