[PATCH 1 of 3] bdiff: use ssize_t everywhere

Yuya Nishihara yuya at tcha.org
Thu Jul 14 11:02:51 EDT 2016


On Thu, 14 Jul 2016 15:51:22 +0200, Maciej Fijalkowski wrote:
> On Thu, Jul 14, 2016 at 1:59 PM, Adrian Buehlmann <adrian at cadifra.com> wrote:
> > On 2016-07-14 13:08, Maciej Fijalkowski wrote:  
> >> # HG changeset patch
> >> # User Maciej Fijalkowski <fijall at gmail.com>
> >> # Date 1468395384 -7200
> >> #      Wed Jul 13 09:36:24 2016 +0200
> >> # Node ID 98a1290a6af89b3146d28388c30dc24b9b7219f8
> >> # Parent  1a1612ddd9721f196690e1fa0831764b43fc6c6e
> >> bdiff: use ssize_t everywhere  
> >                      ^^^^^^^^^^
> > The subject line doesn't match the patch content, as there are still
> > uses of Py_ssize_t in bdiff.c after applying this patch.  
> 
> It uses ssize_t in places that are later copied to a different file. I
> did not want to touch lines that are irrelevant
> as to not introduce changes that are completely unnecessary. What
> would you want instead?

How about "use ssize_t where functions will be cpy-agnostic" ?

I'll review the series tomorrow. Thanks for splitting patches.


More information about the Mercurial-devel mailing list