Re: [PATCH] Add tests for binaryheap.c

Поиск
Список
Период
Сортировка
От Nathan Bossart
Тема Re: [PATCH] Add tests for binaryheap.c
Дата
Msg-id aHf-tbrDaWJPGdTl@nathan
обсуждение исходный текст
Ответ на Re: [PATCH] Add tests for binaryheap.c  (Nathan Bossart <nathandbossart@gmail.com>)
Ответы Re: [PATCH] Add tests for binaryheap.c
Список pgsql-hackers
On Fri, Jun 27, 2025 at 02:02:22PM +0300, Aleksander Alekseev wrote:
>> I'm not sure I see much point in testing both min-heaps and max-heaps.  The
>> only difference between the two is in the comparator, so IMHO the extra
>> tests really only serve to test the test comparator.
> 
> Make sense. Here is the corrected patch.

I trimmed this down some more while trying to keep the coverage the same.

-- 
nathan

Вложения

В списке pgsql-hackers по дате отправления: