Thread (1 message) 1 message, 1 author, 2020-03-03

Re: [PATCH] rebase-interactive.c: silence format-zero-length warnings

From: Junio C Hamano <hidden>
Date: 2020-03-03 14:20:39

Alban Gruin [off-list ref] writes:
quoted
Fixes the following warnings:

rebase-interactive.c: In function ‘edit_todo_list’:
rebase-interactive.c:137:38: warning: zero-length gnu_printf format string [-Wformat-zero-length]
    write_file(rebase_path_dropped(), "");
rebase-interactive.c:144:37: warning: zero-length gnu_printf format string [-Wformat-zero-length]
...
On a tangent: what's wrong with empty format strings?
Those functions that are truly printf-like, such a call would be
no-op and an indication of possible typo ("did you forget a %s or
something?"), I presume.

But many of our functions that take printf-like format strings will
do useful things even when an empty string is given, so the warning
is unwanted.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help