[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,
thanks.

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
account.
-------------- 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