[m-rev.] for review: priority search queue ADT
    Julien Fischer 
    jfischer at opturion.com
       
    Thu Nov 27 14:42:36 AEDT 2014
    
    
  
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'.
- the 'lookup' operation should be named 'search'.
- the 'det_lookup' option should be named 'lookup'
The above renaming are more consistent with the names in the
map module etc.
Cheers,
Julien.
    
    
More information about the reviews
mailing list