[Python-Dev] [Python-checkins] cpython: Issue #9566: Fix a compiler warning in tupleiter_setstate() on Windows x64
Victor Stinner
victor.stinner at gmail.com
Tue Jun 25 00:00:41 CEST 2013
Ah yes correct, it should be better with the following commit:
http://hg.python.org/cpython/rev/3a393fc86b29
Victor
2013/6/24 Jeremy Kloth <jeremy.kloth at gmail.com>:
> On Mon, Jun 24, 2013 at 3:33 PM, victor.stinner
> <python-checkins at python.org> wrote:
>> http://hg.python.org/cpython/rev/6b4d279508a3
>> changeset: 84325:6b4d279508a3
>> user: Victor Stinner <victor.stinner at gmail.com>
>> date: Mon Jun 24 23:31:48 2013 +0200
>> summary:
>> Issue #9566: Fix a compiler warning in tupleiter_setstate() on Windows x64
>> ...
>> static PyObject *
>> tupleiter_setstate(tupleiterobject *it, PyObject *state)
>> {
>> - long index = PyLong_AsLong(state);
>> + Py_ssize_t index = PyLong_AsLong(state);
>
> Actually, this will still lose data when state > MAX_INT (on Windows).
> It should be changed to PyLong_AsSssize_t(state) to ensure consistent
> behavior on all platforms.
More information about the Python-Dev
mailing list