[PATCH STABLE v2] mq: fix qapplied --last and qprev documentation (issue3282)

John Coomes John.Coomes at oracle.com
Wed Feb 29 16:45:35 CST 2012


Matt Mackall (mpm at selenic.com) wrote:
> On Mon, 2012-02-27 at 12:26 -0800, John Coomes wrote:
> > Patrick Mezard (patrick at mezard.eu) wrote:
> > > # HG changeset patch
> > > # User Patrick Mezard <patrick at mezard.eu>
> > > # Date 1330363398 -3600
> > > # Branch stable
> > > # Node ID 673713c957f2b0925df7c22df2d63daaf9470a9e
> > > # Parent  280e834c9d15b0d65a0ee24374671e0cd2147f4e
> > > mq: fix qapplied --last and qprev documentation (issue3282)
> > > 
> > > qapplied --last is qprev not qtop.
> > 
> > I think the behavior (qapplied --last == qprev) is broken, not the
> > documentation.
> 
> History disagrees:
> 
> http://www.selenic.com/hg/rev/c7c2dd7524dd

Broken, misnamed, whatever you want to call it.  Even if it was
intentional, it doesn't make sense.  To make any sense, qapplied
--last (or -1) should behave like qtop.  

The current behavior requires the mental leap of interpreting --last
as meaning 'the one before last' or 'prev'.

-John


More information about the Mercurial-devel mailing list