[PATCH 08 of 10] store: refactor splitting off of "data/" in _hybridencode()

Bryan O'Sullivan bos at serpentine.com
Mon Sep 17 12:58:56 CDT 2012

I've applied the previous patches in this series and pushed them to crew,

However, this one looks wrong.

-    res = 'data/' + '/'.join(auxencode(encodefilename(ndpath)))
> +    res = '/'.join(auxencode(encodefilename(path)))
>      if len(res) > _maxstorepathlen:

As far as I can tell, you've reduced the length of res by 5 bytes here, but
haven't adjusted the conditional immediately afterwards to take this into
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20120917/05d0cb81/attachment.html>

More information about the Mercurial-devel mailing list