Looks good.
--Justin
On Oct 26, 2011, at 10:09 AM, Ben Pfaff wrote:
> ---
> lib/odp-util.c | 7 ++-----
> 1 files changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/lib/odp-util.c b/lib/odp-util.c
> index 33672c8..1f6f33d 100644
> --- a/lib/odp-util.c
> +++ b/lib/odp-util.c
> @@ -33,8 +33,7 @@
> #include "timeval.h"
> #include "util.h"
>
> -static void
> -format_odp_key_attr(const struct nlattr *a, struct ds *ds);
> +static void format_odp_key_attr(const struct nlattr *, struct ds *);
>
> /* The interface between userspace and kernel uses an "OVS_*" prefix.
> * Since this is fairly non-specific for the OVS userspace components,
> @@ -150,8 +149,7 @@ format_odp_userspace_action(struct ds *ds, const struct
> nlattr *attr)
> memcpy(&cookie, &userdata, sizeof cookie);
>
> if (cookie.type == USER_ACTION_COOKIE_CONTROLLER) {
> - ds_put_format(ds, ",controller,length=%"PRIu32,
> - cookie.data);
> + ds_put_format(ds, ",controller,length=%"PRIu32, cookie.data);
> } else if (cookie.type == USER_ACTION_COOKIE_SFLOW) {
> ds_put_format(ds, ",sFlow,n_output=%"PRIu8","
> "vid=%"PRIu16",pcp=%"PRIu8",ifindex=%"PRIu32,
> @@ -279,7 +277,6 @@ odp_flow_key_attr_len(uint16_t type)
> return -1;
> }
>
> -
> static void
> format_generic_odp_key(const struct nlattr *a, struct ds *ds)
> {
> --
> 1.7.2.5
>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
