[ovs-dev] [PATCH] ovsdb-client: Make "dump" print table names in default output format too.

Brad Hall bhall at shell.hallgroup.org
Thu Oct 21 17:57:57 UTC 2010


On Thu, Oct 21, 2010 at 10:49:48AM -0700, Ben Pfaff wrote:
> The "dump" command printed table names in CSV and HTML output formats but
> they were omitted in the default tabular form.  This commit corrects the
> discrepancy.
> 
> Reported-by: Brad Hall <brad at nicira.com>
> ---
>  ovsdb/ovsdb-client.c |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/ovsdb/ovsdb-client.c b/ovsdb/ovsdb-client.c
> index b8a2fa4..326e3b1 100644
> --- a/ovsdb/ovsdb-client.c
> +++ b/ovsdb/ovsdb-client.c
> @@ -441,6 +441,10 @@ table_print_table__(const struct table *table)
>          putchar('\n');
>      }
>  
> +    if (table->caption) {
> +        puts(table->caption);
> +    }
> +
>      if (output_headings) {
>          for (x = 0; x < table->n_columns; x++) {
>              const struct column *column = &table->columns[x];
> -- 
> 1.7.1

Looks good to me.  Thanks for doing this.

-Brad




More information about the dev mailing list