Re: [PATCH 01/11] string_list: print_string_list to use trace_printf
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 6, 2018, 16:52 UTC
- Message-ID
- <xmqqh8j236wj.fsf@gitster-ct.c.googlers.com>
- In-Reply-To
- <20180904230149.180332-2-sbeller@google.com>
Stefan Beller <sbeller@google.com> writes:
> It is a debugging aid, so it should print to the debugging channel.
... and rename it with trace_ prefix.
Use of trace_printf() is nice, as we can control its behavior at runtime ;-)
Show 46 quoted lines
> Signed-off-by: Stefan Beller <sbeller@google.com>
> ---
> string-list.c | 6 +++---
> string-list.h | 4 ++--
> 2 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/string-list.c b/string-list.c
> index 771c4550980..1ebbe1f56ea 100644
> --- a/string-list.c
> +++ b/string-list.c
> @@ -196,13 +196,13 @@ void string_list_clear_func(struct string_list *list, string_list_clear_func_t c
> }
>
>
> -void print_string_list(const struct string_list *p, const char *text)
> +void trace_print_string_list(const struct string_list *p, const char *text)
> {
> int i;
> if ( text )
> - printf("%s\n", text);
> + trace_printf("%s\n", text);
> for (i = 0; i < p->nr; i++)
> - printf("%s:%p\n", p->items[i].string, p->items[i].util);
> + trace_printf("%s:%p\n", p->items[i].string, p->items[i].util);
> }
>
> struct string_list_item *string_list_append_nodup(struct string_list *list,
> diff --git a/string-list.h b/string-list.h
> index ff8f6094a33..5b22560cf19 100644
> --- a/string-list.h
> +++ b/string-list.h
> @@ -114,12 +114,12 @@ void filter_string_list(struct string_list *list, int free_util,
> string_list_each_func_t want, void *cb_data);
>
> /**
> - * Dump a string_list to stdout, useful mainly for debugging
> + * Dump a string_list using the trace_print, useful mainly for debugging
> * purposes. It can take an optional header argument and it writes out
> * the string-pointer pairs of the string_list, each one in its own
> * line.
> */
> -void print_string_list(const struct string_list *p, const char *text);
> +void trace_print_string_list(const struct string_list *p, const char *text);
>
> /**
> * Free a string_list. The `string` pointer of the items will be freed