diff options
author | Victor Stinner <vstinner@python.org> | 2020-09-23 21:25:54 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-09-23 21:25:54 (GMT) |
commit | d67de0a30d76c6a28056bae22fd7d13f2e111b77 (patch) | |
tree | b23fec55f92f6c27707e3125b06ea9eeb1ad2a2e /Objects | |
parent | bbeb223e9a5e9f9374df384efa386b4068a65c0e (diff) | |
download | cpython-d67de0a30d76c6a28056bae22fd7d13f2e111b77.zip cpython-d67de0a30d76c6a28056bae22fd7d13f2e111b77.tar.gz cpython-d67de0a30d76c6a28056bae22fd7d13f2e111b77.tar.bz2 |
bpo-41428: Fix compiler warnings in unionobject.c (GH-22388)
Use Py_ssize_t type rather than int, to store lengths in
unionobject.c. Fix warnings:
Objects\unionobject.c(189,71): warning C4244: '+=':
conversion from 'Py_ssize_t' to 'int', possible loss of data
Objects\unionobject.c(182,1): warning C4244: 'initializing':
conversion from 'Py_ssize_t' to 'int', possible loss of data
Objects\unionobject.c(205,1): warning C4244: 'initializing':
conversion from 'Py_ssize_t' to 'int', possible loss of data
Objects\unionobject.c(437,1): warning C4244: 'initializing':
conversion from 'Py_ssize_t' to 'int', possible loss of data
Diffstat (limited to 'Objects')
-rw-r--r-- | Objects/unionobject.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/Objects/unionobject.c b/Objects/unionobject.c index 0ef7abb..e055a55 100644 --- a/Objects/unionobject.c +++ b/Objects/unionobject.c @@ -179,8 +179,8 @@ exit: static PyObject* flatten_args(PyObject* args) { - int arg_length = PyTuple_GET_SIZE(args); - int total_args = 0; + Py_ssize_t arg_length = PyTuple_GET_SIZE(args); + Py_ssize_t total_args = 0; // Get number of total args once it's flattened. for (Py_ssize_t i = 0; i < arg_length; i++) { PyObject *arg = PyTuple_GET_ITEM(args, i); @@ -434,7 +434,7 @@ _Py_Union(PyObject *args) unionobject* result = NULL; // Check arguments are unionable. - int nargs = PyTuple_GET_SIZE(args); + Py_ssize_t nargs = PyTuple_GET_SIZE(args); for (Py_ssize_t iarg = 0; iarg < nargs; iarg++) { PyObject *arg = PyTuple_GET_ITEM(args, iarg); if (arg == NULL) { |