Remove unused BumpBlockIsValid macro

david-rowley · david-rowley · commit 5a15bdea3b79 · 2024-04-10T11:10:16.000+12:00
The bump allocator was recently added in 29f6a95. 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
diff --git 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
