]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Shorten pg_attribute_always_inline to pg_always_inline
authorTomas Vondra <tomas.vondra@postgresql.org>
Sat, 11 Jul 2026 13:14:50 +0000 (15:14 +0200)
committerTomas Vondra <tomas.vondra@postgresql.org>
Sat, 11 Jul 2026 14:17:16 +0000 (16:17 +0200)
commit1c4b1de888559a47df599dcef356ea7fbf96fd0c
tree969528ba66d5fc0730831728fe3b0a2b8ede0c6a
parent5f14f82280db4b0e49d774362854c9d362106484
Shorten pg_attribute_always_inline to pg_always_inline

The pg_attribute_always_inline macro name is so long it forces pgindent
to format the code in strange ways. Which may incentivize patch authors
to either structure the code in strange ways (e.g. reorder prototypes),
use shorter names, etc. Neither is very desirable for code readability.

This shortens the name by removing the _attribute_ part. It also makes
it more consistent with pg_noinline, which does not have the _attribute_
part either.

Backpatched to all supported branches, to prevent conflicts when
backpatching other fixes. The backbranches however keep both the old and
new macro name, so that existing code keeps working.

Author: Andres Freund <andres@anarazel.de>
Reviewed-by: Peter Geoghegan <pg@bowt.ie>
Reviewed-by: Tomas Vondra <tomas@vondra.me>
Discussion: https://postgr.es/m/bqqdehahpoa36igpictuqyn2s2mexk3t3ehidh2ffd2slb35e5@rzgksuiszgbg
Backpatch-through: 14
15 files changed:
src/backend/access/heap/heapam.c
src/backend/access/transam/xlog.c
src/backend/commands/copyfromparse.c
src/backend/commands/copyto.c
src/backend/executor/execExprInterp.c
src/backend/executor/execTuples.c
src/backend/executor/nodeHashjoin.c
src/backend/executor/nodeSeqscan.c
src/backend/nodes/queryjumblefuncs.c
src/backend/storage/buffer/bufmgr.c
src/backend/utils/adt/json.c
src/backend/utils/cache/catcache.c
src/include/c.h
src/include/executor/execScan.h
src/include/portability/instr_time.h