From 5a15bdea3b791223b4cc708d4953a0086f4332a6 Mon Sep 17 00:00:00 2001 From: David Rowley Date: Wed, 10 Apr 2024 11:10:16 +1200 Subject: [PATCH] Remove unused BumpBlockIsValid macro The bump allocator was recently added in 29f6a959c. Our other allocators have a similar macro to this, but seemingly the version of the macro for those allocators is only used in places where the chunk header is decoded. Since the bump allocator has no chunk header, none of those functions exist for bump therefore macro is unused. Remove it. Reported-by: Peter Eisentraut Discussion: https://postgr.es/m/5f724fb2-96e1-4f36-b65b-47b337ad432e@eisentraut.org --- src/backend/utils/mmgr/bump.c | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/backend/utils/mmgr/bump.c b/src/backend/utils/mmgr/bump.c index 38e8159926..449bd29344 100644 --- a/src/backend/utils/mmgr/bump.c +++ b/src/backend/utils/mmgr/bump.c @@ -100,13 +100,6 @@ struct BumpBlock #define BumpIsValid(set) \ (PointerIsValid(set) && IsA(set, BumpContext)) -/* - * BumpBlockIsValid - * True iff block is valid block of a bump context - */ -#define BumpBlockIsValid(block) \ - (PointerIsValid(block) && BumpIsValid((block)->context)) - /* * We always store external chunks on a dedicated block. This makes fetching * the block from an external chunk easy since it's always the first and only