[PATCH 2 of 9] mercurial: add an extension module for fast dict operations

Bryan O'Sullivan bos at serpentine.com
Mon Mar 25 12:22:34 CDT 2013


On Sun, Mar 24, 2013 at 5:27 PM, Siddharth Agarwal <sid0 at fb.com> wrote:

> +    Extension('mercurial.dicthelpers', ['mercurial/dicthelpers.c']),
>

I have a hazy recollection of a conversation with Matt a few years ago,
from which external packagers have some difficulty with new extension
modules. I have no details, but that's why parsers.c contains a bunch of
random very-much-not-parsers code.

Matt, am I recalling correctly? (And even if I am, is that a valid reason
to continue to contort the code?)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20130325/50884844/attachment.html>


More information about the Mercurial-devel mailing list