Thanks, I applied this to master.
On Fri, May 03, 2013 at 01:48:10PM -0700, Ethan Jackson wrote: > Acked-by: Ethan Jackson <et...@nicira.com> > > On Wed, Feb 13, 2013 at 11:06 PM, Ben Pfaff <b...@nicira.com> wrote: > > We always use unsigned types for port numbers elsewhere, so use one here > > too. > > > > Signed-off-by: Ben Pfaff <b...@nicira.com> > > --- > > lib/ofp-util.c | 2 +- > > lib/ofp-util.h | 2 +- > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/lib/ofp-util.c b/lib/ofp-util.c > > index dd6eed8..2716f82 100644 > > --- a/lib/ofp-util.c > > +++ b/lib/ofp-util.c > > @@ -4519,7 +4519,7 @@ ofputil_parse_key_value(char **stringp, char **keyp, > > char **valuep) > > * will be for Open Flow version 'ofp_version'. Returns message > > * as a struct ofpbuf. Returns encoded message on success, NULL on error */ > > struct ofpbuf * > > -ofputil_encode_dump_ports_request(enum ofp_version ofp_version, int16_t > > port) > > +ofputil_encode_dump_ports_request(enum ofp_version ofp_version, uint16_t > > port) > > { > > struct ofpbuf *request; > > > > diff --git a/lib/ofp-util.h b/lib/ofp-util.h > > index 955886d..5698491 100644 > > --- a/lib/ofp-util.h > > +++ b/lib/ofp-util.h > > @@ -694,7 +694,7 @@ struct ofputil_port_stats { > > }; > > > > struct ofpbuf *ofputil_encode_dump_ports_request(enum ofp_version > > ofp_version, > > - int16_t port); > > + uint16_t port); > > void ofputil_append_port_stat(struct list *replies, > > const struct ofputil_port_stats *ops); > > size_t ofputil_count_port_stats(const struct ofp_header *); > > -- > > 1.7.10.4 > > > > _______________________________________________ > > dev mailing list > > dev@openvswitch.org > > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev