[m-rev.] for review: priority search queue ADT

Julien Fischer jfischer at opturion.com
Thu Nov 27 14:44:22 AEDT 2014



On Thu, 27 Nov 2014, Julien Fischer wrote:

>
> Hi Paul
>
> On Thu, 27 Nov 2014, Paul Bone wrote:
>
>> I've adding this to Mercury at the moment.  Do you have some test code that
>> I can commit to the test suite?
>> 
>> I'm very sorry for making this drag on.  I'll commit the code now and fix 
>> up
>> the few minor issues with a commit of my own.
>
> If you're going to look at this then I won't bother a review of it
> before it's committed.  A few things though:
>
> - the singleton/2 function should be exported.
> - the 'del_min' operation should named 'remove'.

s/remove/remove_min/ there.

Cheers,
Julien.



More information about the reviews mailing list