| Message ID | 20180911184850.24891-1-sbeller@google.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | string-list: remove unused function print_string_list | expand |
Stefan Beller <sbeller@google.com> writes: > The helper function stayed unused for 3 years. A removal of that function I think it stayed unused for more than that before the previous proposal to remove it was written (I do not bother going back to my earlier message that identified which exact commit this was introduced at). It has stayed that way for 3 more years since then. > was proposed before[1], but now time has proven we really do not need the > function. > > [1] https://public-inbox.org/git/1421343725-3973-1-git-send-email-kuleshovmail@gmail.com/ > Signed-off-by: Stefan Beller <sbeller@google.com> I'll add a blank line before the sign-off. Is this an example that our "where is the existing trailer?" code misbehaving? > --- > string-list.c | 10 ---------- > string-list.h | 8 -------- > 2 files changed, 18 deletions(-) > > diff --git a/string-list.c b/string-list.c > index 771c4550980..1f6063f2a27 100644 > --- a/string-list.c > +++ b/string-list.c > @@ -195,16 +195,6 @@ void string_list_clear_func(struct string_list *list, string_list_clear_func_t c > list->nr = list->alloc = 0; > } > > - > -void print_string_list(const struct string_list *p, const char *text) > -{ > - int i; > - if ( text ) > - printf("%s\n", text); > - for (i = 0; i < p->nr; i++) > - printf("%s:%p\n", p->items[i].string, p->items[i].util); > -} > - > struct string_list_item *string_list_append_nodup(struct string_list *list, > char *string) > { > diff --git a/string-list.h b/string-list.h > index ff8f6094a33..18c718c12ce 100644 > --- a/string-list.h > +++ b/string-list.h > @@ -113,14 +113,6 @@ typedef int (*string_list_each_func_t)(struct string_list_item *, void *); > 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 > - * 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); > - > /** > * Free a string_list. The `string` pointer of the items will be freed > * in case the `strdup_strings` member of the string_list is set. The
Junio C Hamano <gitster@pobox.com> writes: > Stefan Beller <sbeller@google.com> writes: > >> The helper function stayed unused for 3 years. A removal of that function > > I think it stayed unused for more than that before the previous > proposal to remove it was written (I do not bother going back to my > earlier message that identified which exact commit this was > introduced at). It has stayed that way for 3 more years since then. > >> was proposed before[1], but now time has proven we really do not need the >> function. >> >> [1] https://public-inbox.org/git/1421343725-3973-1-git-send-email-kuleshovmail@gmail.com/ >> Signed-off-by: Stefan Beller <sbeller@google.com> > > I'll add a blank line before the sign-off. Is this an example that > our "where is the existing trailer?" code misbehaving? I am still curious about this one. In any case, I've reworded the above around "3 years". string-list: remove unused function print_string_list A removal of this helper function was proposed 3 years ago [1]; the function was never used since it was introduced in 2006 back then, and there is no new callers since. Now time has proven we really do not need the function. [1] https://public-inbox.org/git/1421343725-3973-1-git-send-email-kuleshovmail@gmail.com/ Signed-off-by: Stefan Beller <sbeller@google.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> Thanks.
On Tue, Sep 11, 2018 at 12:31 PM Junio C Hamano <gitster@pobox.com> wrote: > > Junio C Hamano <gitster@pobox.com> writes: > > > Stefan Beller <sbeller@google.com> writes: > > > >> The helper function stayed unused for 3 years. A removal of that function > > > > I think it stayed unused for more than that before the previous > > proposal to remove it was written (I do not bother going back to my > > earlier message that identified which exact commit this was > > introduced at). It has stayed that way for 3 more years since then. > > > >> was proposed before[1], but now time has proven we really do not need the > >> function. > >> > >> [1] https://public-inbox.org/git/1421343725-3973-1-git-send-email-kuleshovmail@gmail.com/ > >> Signed-off-by: Stefan Beller <sbeller@google.com> > > > > I'll add a blank line before the sign-off. Is this an example that > > our "where is the existing trailer?" code misbehaving? > > I am still curious about this one. No, it's me who is misbehaving. ;-) While I do have format.signoff set, such that I never forget to sign off on a patch, I have the habit to sign it off manually, in git-gui while editing the commit message so I can see the whole message. > In any case, I've reworded the above around "3 years". > > string-list: remove unused function print_string_list > > A removal of this helper function was proposed 3 years ago [1]; the > function was never used since it was introduced in 2006 back then, > and there is no new callers since. Now time has proven we really do > not need the function. > > [1] https://public-inbox.org/git/1421343725-3973-1-git-send-email-kuleshovmail@gmail.com/ > > Signed-off-by: Stefan Beller <sbeller@google.com> > Signed-off-by: Junio C Hamano <gitster@pobox.com> > > Thanks. Thanks for rewording it. Stefan
Stefan Beller <sbeller@google.com> writes: >> >> >> >> [1] https://public-inbox.org/git/1421343725-3973-1-git-send-email-kuleshovmail@gmail.com/ >> >> Signed-off-by: Stefan Beller <sbeller@google.com> >> > >> > I'll add a blank line before the sign-off. Is this an example that >> > our "where is the existing trailer?" code misbehaving? >> >> I am still curious about this one. > > No, it's me who is misbehaving. ;-) Whew. That lets me worried about one fewer things ;-) Thanks.
diff --git a/string-list.c b/string-list.c index 771c4550980..1f6063f2a27 100644 --- a/string-list.c +++ b/string-list.c @@ -195,16 +195,6 @@ void string_list_clear_func(struct string_list *list, string_list_clear_func_t c list->nr = list->alloc = 0; } - -void print_string_list(const struct string_list *p, const char *text) -{ - int i; - if ( text ) - printf("%s\n", text); - for (i = 0; i < p->nr; i++) - printf("%s:%p\n", p->items[i].string, p->items[i].util); -} - struct string_list_item *string_list_append_nodup(struct string_list *list, char *string) { diff --git a/string-list.h b/string-list.h index ff8f6094a33..18c718c12ce 100644 --- a/string-list.h +++ b/string-list.h @@ -113,14 +113,6 @@ typedef int (*string_list_each_func_t)(struct string_list_item *, void *); 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 - * 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); - /** * Free a string_list. The `string` pointer of the items will be freed * in case the `strdup_strings` member of the string_list is set. The
The helper function stayed unused for 3 years. A removal of that function was proposed before[1], but now time has proven we really do not need the function. [1] https://public-inbox.org/git/1421343725-3973-1-git-send-email-kuleshovmail@gmail.com/ Signed-off-by: Stefan Beller <sbeller@google.com> --- string-list.c | 10 ---------- string-list.h | 8 -------- 2 files changed, 18 deletions(-)