]> 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:45 +0000 (16:17 +0200)
commita474c01c87661f85827736dc8cfc353fd63c6ecf
treea605cdccf9b4891649eea7d5076508a6309f4a32
parente9eaeb04248a8b2cc977440caa1b72174c108c14
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