[PATCH] cli/new: Avoid printing "\033[K" to non-terminal output

Łukasz Stelmach l.stelmach at samsung.com
Fri Mar 10 01:21:27 PST 2017


Tomi Ollila wrote:
>It was <2017-03-09 czw 13:35>, when David Bremner wrote:
>> Łukasz Stelmach <l.stelmach at samsung.com> writes:
>>
>>> ---
>>>  notmuch-new.c | 5 ++---
>>>  1 file changed, 2 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/notmuch-new.c b/notmuch-new.c
>>> index 1321263..44174ad 100644
>>> --- a/notmuch-new.c
>>> +++ b/notmuch-new.c
>>> @@ -905,10 +905,9 @@ print_results (const add_files_state_t *state)
>>>  		state->processed_files == 1 ? "file" : "total files");
>>>  	notmuch_time_print_formatted_seconds (elapsed);
>>>  	if (elapsed > 1)
>>> -	    printf (" (%d files/sec.).\033[K\n",
>>> +	    printf (" (%d files/sec.)",
>>>  		    (int) (state->processed_files / elapsed));
>>> -	else
>>> -	    printf (".\033[K\n");
>>> +	printf(".%s\n", (state->output_is_a_tty) ? "\033[K" : "");
>>>      }
>>
>> Other than missing a space after printf (which I can hopefully remember
>> to add when I commit it, this looks fine).
>
> I looked this briefly a while ago -- there were quite a few \033[K:s
> that were not taken care of... would those have needed to be...

As far as I can tell this is the only one left, that is not under the
condition of output_is_a_tty (at least in notmuch-new).

-- 
Łukasz Stelmach
Samsung R&D Institute Poland
Samsung Electronics
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 472 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20170310/9a9a0205/attachment.sig>


More information about the notmuch mailing list