On 2008-02-21 22:42:51 +0100, Onno Kortmann wrote:
> + if len(args) == 2:
> + old, new=args
> + elif len(args) == 1:
> + if not crt:
> + raise CmdException, "No applied top patch to rename exists."
> + old, [new]=crt, args
I think the style throughout StGit is to have spaces arond operators.
No biggie, though -- I'll fix it up.
> + else: parser.error('incorrect number of arguments')
It looks strange without a line break here, especially since there are
line breaks in the if and elif branches. I'll fix this too if you
don't mind.
--
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
