Serhiy Storchaka added the comment:
This is incorrect. PY_FORMAT_SIZE_T shouldn't be used in format string of
PyErr_Format. On one hand, PyErr_Format understand the z modifier. On other
hand, it is not understand all platform-specific expansions of PY_FORMAT_SIZE_T
(e.g. I).
--
nosy:
Roundup Robot added the comment:
New changeset 6fb1e2ce513a by Victor Stinner in branch '3.4':
Issue #21892, #21893: Partial revert of changeset 4f55e802baf0, PyErr_Format()
http://hg.python.org/cpython/rev/6fb1e2ce513a
New changeset 58cd562e3ef9 by Victor Stinner in branch 'default':
(Merge
STINNER Victor added the comment:
This is incorrect. PY_FORMAT_SIZE_T shouldn't be used in format string of
PyErr_Format.
Oh, I forgot that. Thanks for the careful review Serhiy! Thanks.
--
resolution: - fixed
status: open - closed
___
Python
Changes by Ned Deily n...@acm.org:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21893
___
___
Python-bugs-list mailing list
Roundup Robot added the comment:
New changeset 4f55e802baf0 by Victor Stinner in branch '3.4':
Closes #21892, #21893: Use PY_FORMAT_SIZE_T instead of %zi or %zu to format C
http://hg.python.org/cpython/rev/4f55e802baf0
New changeset 669b43bffd87 by Victor Stinner in branch 'default':
(Merge
STINNER Victor added the comment:
Oh, I forgot the mention John Malmberg in the changelog, sorry :-(
Thanks for your fix John.
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21893
New submission from John Malmberg:
unicodeobject.c is using %zd instead of the PY_FORMAT_SIZE_T as set by
configure.
Comments in pymacconfig.h state that this will cause problems on OS-X 10.4 with
binaries build on OS-X 10.5.
The attached patch unicodeobject.giff changes unicodeobject.c to