Re: patch: Use pg_assume in jsonb_util.c to fix GCC 15 warnings - Mailing list pgsql-hackers

From Tom Lane
Subject Re: patch: Use pg_assume in jsonb_util.c to fix GCC 15 warnings
Date
Msg-id 85365.1752343493@sss.pgh.pa.us
Whole thread Raw
In response to Re: patch: Use pg_assume in jsonb_util.c to fix GCC 15 warnings  (Andres Freund <andres@anarazel.de>)
Responses Re: patch: Use pg_assume in jsonb_util.c to fix GCC 15 warnings
List pgsql-hackers
Andres Freund <andres@anarazel.de> writes:
> On 2025-07-12 13:42:54 -0400, Tom Lane wrote:
>> I don't care for this patch: replacing an Assert with pg_assume just
>> seems like a very bad idea.

> That shouldn't be a problem - pg_assume() is defined to be an Assert() in
> USE_ASSERT_CHECKING builds.

Ah, my bad.  But there's still the question of exactly what reasoning
the compiler is using to arrive at the conclusion that it need not
warn given these assertions, and whether we want to rely on that
reasoning not changing.  I'd prefer to simplify matters.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: patch: Use pg_assume in jsonb_util.c to fix GCC 15 warnings
Next
From: "David E. Wheeler"
Date:
Subject: Re: PATCH: jsonpath string methods: lower, upper, initcap, l/r/btrim, replace, split_part