0001-Remove-bogus-const-qualifier-on-PageGetItem-argument.patch
text/plain
Filename: 0001-Remove-bogus-const-qualifier-on-PageGetItem-argument.patch
Type: text/plain
Part: 0
Message:
Re: some Page/PageData const stuff
Patch
Format: format-patch
Series: patch 0001
Subject: Remove bogus const qualifier on PageGetItem() argument
| File | + | − |
|---|---|---|
| src/include/storage/bufpage.h | 2 | 2 |
From 585627bc317926eb1b690f4647566b4ae26a1453 Mon Sep 17 00:00:00 2001
From: Peter Eisentraut <peter@eisentraut.org>
Date: Sat, 3 Jan 2026 17:58:26 +0100
Subject: [PATCH] Remove bogus const qualifier on PageGetItem() argument
The function ends up casting away the const qualifier, so it was a
lie. No callers appear to rely on the const qualifier on the
argument, so the simplest solution is to just remove it.
---
src/include/storage/bufpage.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/include/storage/bufpage.h b/src/include/storage/bufpage.h
index 52989e5e535..ae3725b3b81 100644
--- a/src/include/storage/bufpage.h
+++ b/src/include/storage/bufpage.h
@@ -351,12 +351,12 @@ PageValidateSpecialPointer(const PageData *page)
* The semantics may change in the future.
*/
static inline void *
-PageGetItem(const PageData *page, const ItemIdData *itemId)
+PageGetItem(PageData *page, const ItemIdData *itemId)
{
Assert(page);
Assert(ItemIdHasStorage(itemId));
- return (void *) (((const char *) page) + ItemIdGetOffset(itemId));
+ return (char *) page + ItemIdGetOffset(itemId);
}
/*
--
2.52.0