[issue22156] Fix compiler warnings "comparison between signed and unsigned integers"

STINNER Victor added the comment: > I'm not sure about _tracemalloc.c. May be make MAX_NFRAME an int would be > simpler? Ok, I agree, I changed it. -- ___ Python tracker ___ __

[issue22156] Fix compiler warnings "comparison between signed and unsigned integers"

Roundup Robot added the comment: New changeset a318a5ab91fe by Victor Stinner in branch 'default': Issue #22156: simplify _tracemalloc.c, use an int for the MAX_NFRAME constant http://hg.python.org/cpython/rev/a318a5ab91fe -- ___ Python tracker

[issue22156] Fix compiler warnings "comparison between signed and unsigned integers"

Serhiy Storchaka added the comment: I'm not sure about _tracemalloc.c. May be make MAX_NFRAME an int would be simpler? Other changes LGTM. -- ___ Python tracker ___

[issue22156] Fix compiler warnings "comparison between signed and unsigned integers"

STINNER Victor added the comment: Ok, I fixed many warnings in this issue. I will open new issues for other warnings. -- resolution: -> fixed status: open -> closed ___ Python tracker

[issue22156] Fix compiler warnings "comparison between signed and unsigned integers"

Roundup Robot added the comment: New changeset e831a98b3f43 by Victor Stinner in branch 'default': Issue #22156: Fix some "comparison between signed and unsigned integers" http://hg.python.org/cpython/rev/e831a98b3f43 -- ___ Python tracker

[issue22156] Fix compiler warnings "comparison between signed and unsigned integers"

Changes by STINNER Victor : -- title: Fix compiler warnings -> Fix compiler warnings "comparison between signed and unsigned integers" ___ Python tracker ___ ___

[issue22156] Fix compiler warnings

STINNER Victor added the comment: I created another more specific issue: #22207, "Test for integer overflow on Py_ssize_t: explicitly cast to size_t". -- ___ Python tracker ___

[issue22156] Fix compiler warnings

STINNER Victor added the comment: I created a specific issue for PyThread_create_key() of Python/thread_pthread.h: issue #22206. -- ___ Python tracker ___ __

[issue22156] Fix compiler warnings

Roundup Robot added the comment: New changeset 9b84ff16edd4 by Victor Stinner in branch 'default': Issue #22156: Fix "comparison between signed and unsigned integers" compiler http://hg.python.org/cpython/rev/9b84ff16edd4 New changeset a0b38f4eb79e by Victor Stinner in branch 'default': Issue #2

[issue22156] Fix compiler warnings

STINNER Victor added the comment: > Victor, your patch fixes most of the pesky warnings. However you left one > warning ... Yes, my patch is uncomplete. Don't hesitate to post a new patch or complete mine. I'm in holliday, I'm not going to update it before a few weeks. --

[issue22156] Fix compiler warnings

Vajrasky Kok added the comment: Victor, your patch fixes most of the pesky warnings. However you left one warning left (at least in Mac OS X 10.9.4). Objects/unicodeobject.c:4831:43: warning: comparison of integers of different signs: 'unsigned long' and 'long' [-Wsign-compare] if (PY_SSIZ

[issue22156] Fix compiler warnings

Changes by Serhiy Storchaka : -- components: +Extension Modules, Interpreter Core nosy: +serhiy.storchaka stage: -> patch review type: -> compile error ___ Python tracker ___ _

[issue22156] Fix compiler warnings

New submission from STINNER Victor: The issue #22110 enabled more compiler warnings. Attached patch tries to fix most of them on Linux. -- files: fix_warnings.patch keywords: patch messages: 224924 nosy: haypo, neologix priority: normal severity: normal status: open title: Fix compiler