On Fri, 23 Feb 2018 18:54:53 -0800, Gregory Szorc wrote:
> Speaking of __int__, it is only defined in one place in the repo: on
> context.basectx. I think we should consider removing it: it is too much
> coercion magic for my liking.

+1 for removing int(ctx) and '%d' % ctx. Actually it doesn't always return
an int because rev() may be None.
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to