In a later patch we want to use more of the column_options members at places, where we do actual output, so it will be handy to have the whole struct around in `display_plain`. Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx> --- column.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/column.c b/column.c index d55ead18ef..4851b9aa04 100644 --- a/column.c +++ b/column.c @@ -109,12 +109,12 @@ static void shrink_columns(struct column_data *data) /* Display without layout when not enabled */ static void display_plain(const struct string_list *list, - const char *indent, const char *nl) + const struct column_options *opts) { int i; for (i = 0; i < list->nr; i++) - printf("%s%s%s", indent, list->items[i].string, nl); + printf("%s%s%s", opts->indent, list->items[i].string, opts->nl); } /* Print a cell to stdout with all necessary leading/traling space */ @@ -201,12 +201,14 @@ void print_columns(const struct string_list *list, unsigned int colopts, nopts.padding = opts ? opts->padding : 1; nopts.width = opts && opts->width ? opts->width : term_columns() - 1; if (!column_active(colopts)) { - display_plain(list, "", "\n"); + nopts.indent = ""; + nopts.nl = "\n"; + display_plain(list, &nopts); return; } switch (COL_LAYOUT(colopts)) { case COL_PLAIN: - display_plain(list, nopts.indent, nopts.nl); + display_plain(list, &nopts); break; case COL_ROW: case COL_COLUMN: -- 2.12.2.511.g2abb8caf66