Remove unused BumpBlockIsValid macro
authorDavid Rowley <drowley@postgresql.org>
Tue, 9 Apr 2024 23:10:16 +0000 (11:10 +1200)
committerDavid Rowley <drowley@postgresql.org>
Tue, 9 Apr 2024 23:10:16 +0000 (11:10 +1200)
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

index 38e8159926227dc11b7393e853b8aca249897e93..449bd293448c54fe109a5ab1df2627c4b764c751 100644 (file)
@@ -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