On 2009-01-14 22:59:30 +0000, Catalin Marinas wrote:
> + opt('-f', '--files', action = 'append',
> + short = 'Only fold the given files'),
The long form should probably be "--file", since you only list one
file for every flag (and since you call it "--file" in the last hunk).
And the help text could be something like
Only fold the given file (for multiple files, use -f more than once)
> + raise CmdException, 'Patch folding failed'
This is not important, but I believe the recommended syntax for
raising exceptions nowadays is
CmdException('Patch folding failed')
since that's what'll continue to work in Python 3, or something like
that. (I see you already use it in the multi-line case in the other
patch, where this syntax is clearly the better choice.)
--
Karl Hasselström, kha@xxxxxxxxxxx
www.treskal.com/kalle
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
