[PATCH 1 of 7 resend] store: implement lowerencode in C

Bryan O'Sullivan bos at serpentine.com
Fri Dec 14 12:21:07 CST 2012


On Fri, Dec 14, 2012 at 2:34 AM, Adrian Buehlmann <adrian at cadifra.com>wrote:

> May I ask a stupid question: Why are you resending this?
>

It needed a small bugfix to do with our friend Py_ssize_t, but also it's
been a month since I submitted the original patchset.


> I'm currently still quite a bit puzzled by what you pushed with
> 736f1c09f321.
>

It's more or less a no-op for now, and since it's not a piece of core
functionality and wasn't going to do any harm upstream, I judged it as no
harm to push it. Think of it as a balancing act between trying to get easy
things done and getting tricky things reviewed, especially given that we
are extremely bandwidth-deprived on the reviewing side.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20121214/a18a91f6/attachment.html>


More information about the Mercurial-devel mailing list