amcheck: Remove unused IndexCheckableCallback typedef
Hi,
Commit d70b17636dd introduced the IndexCheckableCallback typedef for
a callback function, but it appears to be unused.
From the discussion in [1]/messages/by-id/45AC9B0A-2B45-40EE-B08F-BDCF5739D1E1@yandex-team.ru, it seems this typedef was originally used as
a function argument in early versions of the patch. However,
the argument was removed in v23, and the typedef was accidentally left behind.
The attached patch removes this unused typedef to clean up the dead code.
Thoughts?
Regards,
[1]: /messages/by-id/45AC9B0A-2B45-40EE-B08F-BDCF5739D1E1@yandex-team.ru
--
Fujii Masao
NTT DATA Japan Corporation
Attachments:
v1-0001-amcheck-Remove-unused-IndexCheckableCallback-type.patchtext/plain; charset=UTF-8; name=v1-0001-amcheck-Remove-unused-IndexCheckableCallback-type.patchDownload
From 2d38d5cbb3d764f033d2e93b3eba1e4b735a6c28 Mon Sep 17 00:00:00 2001
From: Fujii Masao <fujii@postgresql.org>
Date: Fri, 4 Jul 2025 15:28:24 +0900
Subject: [PATCH v1] amcheck: Remove unused IndexCheckableCallback typedef.
Commit d70b17636dd introduced the IndexCheckableCallback typedef for
a callback function, but it was never used. This commit removes
the unused typedef to clean up dead code.
---
contrib/amcheck/verify_common.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/contrib/amcheck/verify_common.h b/contrib/amcheck/verify_common.h
index e78adb68808..42ef9c20fe2 100644
--- a/contrib/amcheck/verify_common.h
+++ b/contrib/amcheck/verify_common.h
@@ -16,8 +16,7 @@
#include "utils/relcache.h"
#include "miscadmin.h"
-/* Typedefs for callback functions for amcheck_lock_relation_and_check */
-typedef void (*IndexCheckableCallback) (Relation index);
+/* Typedef for callback function for amcheck_lock_relation_and_check */
typedef void (*IndexDoCheckCallback) (Relation rel,
Relation heaprel,
void *state,
--
2.49.0
On 4 Jul 2025, at 10:50, Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
From the discussion in [1], it seems this typedef was originally used as
a function argument in early versions of the patch. However,
the argument was removed in v23, and the typedef was accidentally left behind.
Yes, your analysis is correct. We could make common checking function and argument was not needed.
The attached patch removes this unused typedef to clean up the dead code.
Thoughts?
Looks good to me. Thanks for fixing this!
Best regards, Andrey Borodin.
On 2025/07/04 17:25, Andrey Borodin wrote:
On 4 Jul 2025, at 10:50, Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
From the discussion in [1], it seems this typedef was originally used as
a function argument in early versions of the patch. However,
the argument was removed in v23, and the typedef was accidentally left behind.Yes, your analysis is correct. We could make common checking function and argument was not needed.
The attached patch removes this unused typedef to clean up the dead code.
Thoughts?Looks good to me. Thanks for fixing this!
Thanks for the review! I've pushed the patch.
Regards,
--
Fujii Masao
NTT DATA Japan Corporation