[issue21442] Win32 compiler warning in PyBytes_Concat

2014-05-06 Thread Zachary Ware
Zachary Ware added the comment: Declaring oldsize as Py_ssize_t instead of size_t at Objects/bytesobject.c:2812 takes care of the warning, but I'm not comfortable saying that's a correct change. -- ___ Python tracker rep...@bugs.python.org

[issue21442] Win32 compiler warning in PyBytes_Concat

2014-05-06 Thread Zachary Ware
Changes by Zachary Ware zachary.w...@gmail.com: -- nosy: +tim.golden ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue21442 ___ ___ Python-bugs-list

[issue21442] Win32 compiler warning in PyBytes_Concat

2014-05-06 Thread Tim Golden
Tim Golden added the comment: Well PyBytes_GET_SIZE is documented as returning a Py_ssize_t so I'd say it's perfectly valid. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue21442 ___

[issue21442] Win32 compiler warning in PyBytes_Concat

2014-05-06 Thread Roundup Robot
Roundup Robot added the comment: New changeset 6234f4caba57 by Zachary Ware in branch 'default': Issue #21442: Fix MSVC compiler warning introduced by issue21377. http://hg.python.org/cpython/rev/6234f4caba57 -- nosy: +python-dev ___ Python tracker

[issue21442] Win32 compiler warning in PyBytes_Concat

2014-05-06 Thread Zachary Ware
Zachary Ware added the comment: I should have looked a little harder, thanks Tim! -- assignee: - zach.ware resolution: - fixed stage: needs patch - resolved status: open - closed ___ Python tracker rep...@bugs.python.org

[issue21442] Win32 compiler warning in PyBytes_Concat

2014-05-05 Thread Zachary Ware
New submission from Zachary Ware: Issue #21377 introduced a compiler warning on Windows: ..\Objects\bytesobject.c(2824): warning C4018: '' : signed/unsigned mismatch [P:\ath\to\cpython\PCbuild\pythoncore.vcxproj] -- components: Build, Windows messages: 217942 nosy: haypo, nikratio,