Looks good,

Ethan

On Thu, Sep 29, 2011 at 18:44, Justin Pettit <jpet...@nicira.com> wrote:
> When no remote MPIDs were found, the output would print an extra newline.
> If multiple remote MPIDs were found, the lines would run together.  This
> commit cleans things up a bit by just printing each item on its own line
> without any blank lines.
> ---
>  lib/cfm.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/lib/cfm.c b/lib/cfm.c
> index ff401dc..3f3dcd2 100644
> --- a/lib/cfm.c
> +++ b/lib/cfm.c
> @@ -533,12 +533,11 @@ cfm_print_details(struct ds *ds, const struct cfm *cfm)
>     ds_put_format(ds, "\tnext fault check: %lldms\n",
>                   timer_msecs_until_expired(&cfm->fault_timer));
>
> -    ds_put_cstr(ds, "\n");
>     HMAP_FOR_EACH (rmp, node, &cfm->remote_mps) {
>         ds_put_format(ds, "Remote MPID %"PRIu64":%s\n",
>                       rmp->mpid,
>                       rmp->rdi ? " rdi" : "");
> -        ds_put_format(ds, "\trecv since check: %s",
> +        ds_put_format(ds, "\trecv since check: %s\n",
>                       rmp->recv ? "true" : "false");
>     }
>  }
> --
> 1.7.1
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to