[PATCH 11 of 11] merge: don't fiddle with name lookups or i18n in hot loops

Bryan O'Sullivan bos at serpentine.com
Mon Feb 11 19:02:18 CST 2013


On Sun, Feb 10, 2013 at 3:25 AM, Idan Kamara <idankk86 at gmail.com> wrote:

> No, he didn't. And had he given me the chance to reply to his last
> response, I would have shown it:
>

Yep, that's a bug. But it's just a bug - I fixed another one in the new
parallel code just a little while ago.


> And the only reason the sys.exit I had a problem with is
> 'working' is thanks to this:
>
> http://selenic.com/repo/hg/file/0027a5cec9d0/mercurial/dispatch.py#l201
>

Correct, and that's deliberate. Perhaps you could tell me what you'd rather
see, because I obviously think the use of sys.exit in this context is
reasonable based on what I currently know.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20130211/6677a735/attachment.html>


More information about the Mercurial-devel mailing list