Re: [PATCH 1/9] test-mergesort: use strbuf_getline()
- From
René Scharfe <l.s.r@web.de>
- Date
- Oct 2, 2021, 16:56 UTC
- Message-ID
- <0bff46b8-fd45-c4e5-e8b3-60e68ba7c2b9@web.de>
- In-Reply-To
- <87fstjpzdu.fsf@evledraar.gmail.com>
Am 02.10.21 um 11:08 schrieb Ævar Arnfjörð Bjarmason:
Show 10 quoted lines
>
> On Fri, Oct 01 2021, René Scharfe wrote:
>
>> while (lines) {
>> - printf("%s", lines->text);
>> + puts(lines->text);
>> lines = lines->next;
>
> Aside: I wonder if we should have a coccicheck for that (not as part of
> this series), but maybe it would generate too much noise.To replace printf("%s\n", s) with puts(s)? I considered such a semantic patch before as well. It would effectively forbid that specific printf usage. And why shouldn't it? puts(3) is easier to use and its call is shorter.
But puts(3) is also confusing: It adds a trailing newline, but its sibling fputs(3) doesn't. And it would look weird in the middle of a run of printf calls.
Clang already does the replacement with -O1 and GCC even with -O0, so there is no further performance improvement or object text size reduction to be gained.
And so I dropped the idea.
René