Обсуждение: Remove redundant variable pageSize in gistinitpage

Hi,

In gistinitpage, pageSize variable looks redundant, instead we could
just pass BLCKSZ. This will be consistent with its peers
BloomInitPage, brin_page_init and SpGistInitPage. Attaching a small
patch. Thoughts?

With Regards,
Bharath Rupireddy.
EnterpriseDB: http://www.enterprisedb.com

On 26.04.21 05:12, Bharath Rupireddy wrote:
> In gistinitpage, pageSize variable looks redundant, instead we could
> just pass BLCKSZ. This will be consistent with its peers
> BloomInitPage, brin_page_init and SpGistInitPage. Attaching a small
> patch. Thoughts?

Committed.

This code was new in this form in PG14 
(16fa9b2b30a357b4aea982bd878ec2e5e002dbcc), so it made sense to clean it 
up now.