The issue #22110 enabled more compiler warnings. Attached patch fixes some of them in the Modules/ subdirectory.

New changeset 05e8f92b58ff by Victor Stinner in branch 'default':
Issue #22218: Fix "comparison between signed and unsigned integers" warnings in
http://hg.python.org/cpython/rev/05e8f92b58ff

You are very fast.

New changeset ca5eef14c1ab by Victor Stinner in branch 'default':
Issue #22218: Fix "comparison between signed and unsigned integers" warnings in
http://hg.python.org/cpython/rev/ca5eef14c1ab

New changeset 2f2c1816d0c7 by Victor Stinner in branch 'default':
Issue #22218: Fix "comparison between signed and unsigned integers" warning in
http://hg.python.org/cpython/rev/2f2c1816d0c7

> You are very fast.

Oh sorry. In fact, I didn't expect a review. I just created an issue to put an issue number in the commit changelog and give a little bit more context to my changes.

Hi Victor,

Here is a small patch for the unicodeobject.c file.

I am not sure if it's the correct solution to this problem, but for me, the size parameter should be a unsigned long and not a signed.

a patch for 3.6 for the unicodeobject.c file

fix_warning_unicodeobject.diff and issue22218-2.patch don't seem to be needed anymore. The code is now:

    if (PY_SSIZE_T_MAX / (Py_ssize_t)sizeof(wchar_t) < (size + 1))
        return NULL;

I close the issue. More specific issues must be opened if you still want to fix similar issues.