Hi Nathan,

> Adding some tests here seems like a good idea.
>
> 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.

-- 
Best regards,
Aleksander Alekseev

Attachment: v2-0001-Add-tests-for-binaryheap.c.patch
Description: Binary data

Reply via email to