[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