Re: A compiling warning in jsonb_populate_record_valid

Поиск
Список
Период
Сортировка
От Amit Langote
Тема Re: A compiling warning in jsonb_populate_record_valid
Дата
Msg-id CA+HiwqEW6g8VozHSuHWz5TPh2=coSUYTrFS5fnHL+2_xqn=1rQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: A compiling warning in jsonb_populate_record_valid  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: A compiling warning in jsonb_populate_record_valid  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
 On Thu, Jan 25, 2024 at 23:57 Tom Lane <tgl@sss.pgh.pa.us> wrote:
Amit Langote <amitlangote09@gmail.com> writes:
> On Thu, Jan 25, 2024 at 2:59 PM Richard Guo <guofenglinux@gmail.com> wrote:
>> I came across a warning when building master (a044e61f1b) on old GCC
>> (4.8.5).

> Will apply the attached, which does this:

> -   return BoolGetDatum(!SOFT_ERROR_OCCURRED(&escontext));
> +   return BoolGetDatum(!escontext.error_occurred);

-1 please.  We should not break that abstraction for the sake
of ignorable warnings on ancient compilers.

Ignoring the warning was my 1st thought too, because an old discussion I found about the warning was too old (2011).  The style I adopted in my “fix” is used in a few other places too, so I thought I might as well go for it.

Anyway, I’m fine with reverting.

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

Предыдущее
От: Robert Haas
Дата:
Сообщение: Re: cleanup patches for incremental backup
Следующее
От: Tom Lane
Дата:
Сообщение: Re: A compiling warning in jsonb_populate_record_valid