From 3e4957dcb0ed5d2512ed2d9fbd55a3c1df23bc1e Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Fri, 28 Aug 2026 23:08:56 +0200 Subject: [PATCH 2/5] Remove unused global variable [test_resowner] src/test/modules/test_resowner/test_resowner_many.c:49:29: error: variable 'current_release_phase' set but not used [-Werror,-Wunused-but-set-global] This was apparently never used (initial commit b8bff07daa8). --- src/test/modules/test_resowner/test_resowner_many.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/test/modules/test_resowner/test_resowner_many.c b/src/test/modules/test_resowner/test_resowner_many.c index 4962b371bdc..40a3948ae04 100644 --- a/src/test/modules/test_resowner/test_resowner_many.c +++ b/src/test/modules/test_resowner/test_resowner_many.c @@ -43,10 +43,9 @@ typedef struct } ManyTestResource; /* - * Current release phase, and priority of last call to the release callback. + * Current release priority of last call to the release callback. * This is used to check that the resources are released in correct order. */ -static ResourceReleasePhase current_release_phase; static uint32 last_release_priority = 0; /* prototypes for local functions */ @@ -272,18 +271,15 @@ test_resowner_many(PG_FUNCTION_ARGS) /* Start releasing */ elog(NOTICE, "releasing resources before locks"); - current_release_phase = RESOURCE_RELEASE_BEFORE_LOCKS; last_release_priority = 0; ResourceOwnerRelease(resowner, RESOURCE_RELEASE_BEFORE_LOCKS, false, false); Assert(GetTotalResourceCount(before_kinds, nkinds) == 0); elog(NOTICE, "releasing locks"); - current_release_phase = RESOURCE_RELEASE_LOCKS; last_release_priority = 0; ResourceOwnerRelease(resowner, RESOURCE_RELEASE_LOCKS, false, false); elog(NOTICE, "releasing resources after locks"); - current_release_phase = RESOURCE_RELEASE_AFTER_LOCKS; last_release_priority = 0; ResourceOwnerRelease(resowner, RESOURCE_RELEASE_AFTER_LOCKS, false, false); Assert(GetTotalResourceCount(before_kinds, nkinds) == 0); -- 2.55.0