No need to squash. This looks fine, I'll merge as soon as I verify the
build status (though I don't anticipate any problems) Thanks.


On Fri, May 10, 2013 at 2:28 PM, Alok Singhal <notifications@github.com>wrote:

> In Cython/Includes/cpython/unicode.pxd:
>
> > @@ -377,7 +377,7 @@ cdef extern from *:
> >      # consumed is not NULL, PyUnicode_DecodeMBCSStateful() will not
> >      # decode trailing lead byte and the number of bytes that have been
> >      # decoded will be stored in consumed. New in version 2.5.
> > -    object PyUnicode_DecodeMBCSStateful(char *s, int size, char *errors, int *consumed)
> > +    object PyUnicode_DecodeMBCSStateful(char *s, Py_ssize_t size, char *errors, Py_ssize_t *consumed)
>
> Thanks for the comment. I have now fixed this. I have added another commit
> on top of the original pull request fixing this. Should I "squash" the two
> commits together? Not sure if github will still consider the new branch to
> be a part of this pull request in that case though.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cython/cython/pull/224/files#r4179820>
> .
>