Re: [PATCH] Add tests for binaryheap.c - Mailing list pgsql-hackers

From Aleksander Alekseev
Subject Re: [PATCH] Add tests for binaryheap.c
Date
Msg-id CAJ7c6TMxK4kN-5A13NThatFEBsTExsL4m9zE++UE69COZQLj2w@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Add tests for binaryheap.c  (Nathan Bossart <nathandbossart@gmail.com>)
Responses Re: [PATCH] Add tests for binaryheap.c
List pgsql-hackers
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.

Attachment

pgsql-hackers by date:

Previous
From: Álvaro Herrera
Date:
Subject: Re: [Feature Request] Support INSERT ... FROM ... TO for date ranges
Next
From: Amit Langote
Date:
Subject: Re: generic plans and "initial" pruning