[m-rev.] Proposed changes to pqueue.m

Julien Fischer jfischer at opturion.com
Tue Nov 19 11:36:51 AEDT 2013


On Tue, Nov 19, 2013 at 8:57 AM, Paul Bone <paul at bone.id.au> wrote:

> On Tue, Nov 19, 2013 at 01:21:50AM +1100, Julien Fischer wrote:
> >
> > Hi,
> >
> > On Mon, 18 Nov 2013, Paul Bone wrote:
> >
> >>>  @@ -47,6 +51,15 @@
> >>>   :- pred pqueue.insert(K::in, V::in, pqueue(K, V)::in, pqueue(K,
> V)::out)
> >>>       is det.
> >>>
> >>>  +    % Extract the smallest item from the priority queue without
> removing
> >>> it.
> >>>  +    % Fails if the priority queue is empty.
> >>>  +    %
> >>>  +:- pred pqueue.peek(pqueue(K, V)::in, V::out) is semidet.
> >>>  +
> >>>  +    % As above, but calls error/1 if the priority queue is empty.
> >>>  +:- func pqueue.det_peek(pqueue(K, V)) = V.
> >>>  +:- pred pqueue.det_peek(pqueue(K, V)::in, V::out) is det.
> >>>  +
> >>>       % Remove the smallest item from the priority queue.
> >>>       % Fails if the priority queue is empty.
> >>>       %
> >>
> >> How long are lines in your text editor?  We like to limit lines to 76
> >> columns so taht a diff plus an e-mail reply doesn't cause wrapping.
> >
> > Do we?  According to the Mercury project's coding standard we limit them
> > to 79 characters.  And indeed, that's what most of the code does.
>
> Can we change it to 76?  I've been assuming 76 the whole time.


No.

Cheers,
Julien.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20131119/174d90e8/attachment.html>


More information about the reviews mailing list