Horton

Re: [PATCH] Add tests for binaryheap.c

Aleksander Alekseev <aleksander@tigerdata.com>

View thread View on postgr.es
From: Aleksander Alekseev <aleksander@tigerdata.com>
To: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Cc: Nathan Bossart <nathandbossart@gmail.com>
Date: 2025-07-17T12:03:55Z
Lists: pgsql-hackers

Attachments

Hi Nathan,

> > Make sense. Here is the corrected patch.
>
> I trimmed this down some more while trying to keep the coverage the same.

LGTM except for a slight issue with double semicolons at
test_binaryheap.c:160. Here is the updated patch.

Commits

  1. Add a test harness for the binary heap code.