Re: [PATCH] Use binaryheap_* macro where appropriate

Nathan Bossart <nathandbossart@gmail.com>

From: Nathan Bossart <nathandbossart@gmail.com>
To: Aleksander Alekseev <aleksander@timescale.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-06-26T20:16:35Z
Lists: pgsql-hackers
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



Commits

  1. Make more use of binaryheap_empty() and binaryheap_size().