Re: Fix possible bogus array out of bonds (src/backend/access/brin/brin_minmax_multi.c)

Поиск
Список
Период
Сортировка
От David Rowley
Тема Re: Fix possible bogus array out of bonds (src/backend/access/brin/brin_minmax_multi.c)
Дата
Msg-id CAApHDvp2THseLvCc+TcYFBC7FKHpHTs1JyYmd2JghtOVhb5WGA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Fix possible bogus array out of bonds (src/backend/access/brin/brin_minmax_multi.c)  (Ranier Vilela <ranier.vf@gmail.com>)
Список pgsql-hackers
On Tue, 6 Sept 2022 at 23:25, Ranier Vilela <ranier.vf@gmail.com> wrote:
> But first, I would like to continue with this correction of using strings.
> In the following cases:
> fprintf -> fputs -> fputc
> printf -> puts -> putchar
>
> There are many occurrences, do you think it would be worth the effort?

I'm pretty unexcited about that. Quite a bit of churn and adding
another precedent that we currently have no good way to enforce or
maintain.

In addition to that, puts() is a fairly seldom used function, which
perhaps is because it's a bit quirky and appends a \n to the end of
the string. I'm just imagining all the bugs where we append an extra
newline. But, feel free to open another thread about it and see if you
can drum up any support.

David



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Justin Pryzby
Дата:
Сообщение: Re: New docs chapter on Transaction Management and related changes
Следующее
От: Reid Thompson
Дата:
Сообщение: Re: Add tracking of backend memory allocated to pg_stat_activity