On Wed, Apr 29, 2009 at 8:40 PM, Trey Roessig <[email protected]> wrote:
> On Wed, Apr 29, 2009 at 3:48 PM, Steve Borho <[email protected]> wrote:
>> The third parameter actually needed to be an empty list, []
>> so I fixed up your patch and pushed it.
>
> Really?  The third entry is "default" which, based on the 'serve' pattern
> a few lines down, I thought would be ok with an empty string.  'Serve'
> may need to be fixed as well.

The default value is entirely defined by how the command tries to use
the  supplied argument.  For status, it expects a list of revisions so
you can specify things like:

hgtk status -r 10 -r 12    # show changes between revs 10 and 12

> Actually, looking at those lines makes me think almost every one of
> them might bomb, because most only have three entries.  Since only
> status had problems, I assume I don't have the whole picture.

I think you're confusing the top level dictionary format of:

'cmdname':  (func, [args], helpstr)

With the format of the args list:

[ (short, long, default, help), ... ]

--
Steve Borho

------------------------------------------------------------------------------
Register Now & Save for Velocity, the Web Performance & Operations 
Conference from O'Reilly Media. Velocity features a full day of 
expert-led, hands-on workshops and two days of sessions from industry 
leaders in dedicated Performance & Operations tracks. Use code vel09scf 
and Save an extra 15% before 5/3. http://p.sf.net/sfu/velocityconf
_______________________________________________
Tortoisehg-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/tortoisehg-develop

Reply via email to