Request a Demo
Home   >   mailing lists

Re: [PATCH] Use binaryheap_* macro where appropriate - Mailing list pgsql-hackers

From Nathan Bossart
Subject Re: [PATCH] Use binaryheap_* macro where appropriate
Date
Msg-id aF2qowqbG1FW8lgF@nathan
Whole thread Raw
In response to [PATCH] Use binaryheap_* macro where appropriate  (Aleksander Alekseev <aleksander@timescale.com>)
List pgsql-hackers
Tree view 
On Thu, Jun 26, 2025 at 01:18:10PM +0300, Aleksander Alekseev wrote:
> I noticed several places where we access `struct binaryheap` directly
> instead of using binaryheap_size() and binaryheap_empty() which
> doesn't seem right. Here is the patch.

Seems reasonable.  I'll plan on committing this once v19 is open for
business.

-- 
nathan

pgsql-hackers by date:

Previous
From: Jianghua Yang
Date:
Subject: [PATCH] Use DatumGetUInt32() for dsm_attach() in test_shm_mq_main()
Next
From: Nathan Bossart
Date:
Subject: Re: [PATCH] Use DatumGetUInt32() for dsm_attach() in test_shm_mq_main()