public inbox for [email protected]
help / color / mirror / Atom feedFrom: Peter Eisentraut <[email protected]>
To: [email protected]
Subject: pgsql: Change remaining StaticAssertStmt() to StaticAssertDecl()
Date: Mon, 16 Feb 2026 08:38:03 +0000
Message-ID: <[email protected]> (raw)
Change remaining StaticAssertStmt() to StaticAssertDecl()
This completes the work started by commit 75f49221c22.
In basebackup.c, changing the StaticAssertStmt to StaticAssertDecl
results in having the same StaticAssertDecl() in 2 functions. So, it
makes more sense to move it to file scope instead.
Also, as it depends on some computations based on 2 tar blocks, define
TAR_NUM_TERMINATION_BLOCKS.
In deadlock.c, change the StaticAssertStmt to StaticAssertDecl and
keep it in the function scope. Add new braces to avoid warning from
-Wdeclaration-after-statement.
In aset.c, change the StaticAssertStmt to StaticAssertDecl and move it
to file scope.
Finally, update the comments in c.h a bit.
Author: Bertrand Drouvot <[email protected]>
Co-authored-by: Peter Eisentraut <[email protected]>
Discussion: https://www.postgresql.org/message-id/aYH6ii46AvGVCB84%40ip-10-97-1-34.eu-west-3.compute.internal
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/d50c86e743755e7ea91e5980f09f8575e0cb338b
Modified Files
--------------
src/backend/backup/basebackup.c | 17 +++++++++--------
src/backend/storage/lmgr/deadlock.c | 12 +++++++-----
src/backend/utils/mmgr/aset.c | 10 ++++------
src/include/c.h | 24 +++++++++++++++++-------
4 files changed, 37 insertions(+), 26 deletions(-)
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected]
Subject: Re: pgsql: Change remaining StaticAssertStmt() to StaticAssertDecl()
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox