diff options
| author |   Serhiy Storchaka <storchaka@gmail.com> | 2017-03-12 14:43:12 +0200 |
|---|---|---|
| committer |   GitHub <noreply@github.com> | 2017-03-12 14:43:12 +0200 |
| commit | 783d0c1a1c723733adcdf4249240246fc35a5bcb (patch) | |
| tree | 15745c18439463475dbc63be52fbad5fff4f6868 /Modules/selectmodule.c | |
| parent | bpo-15695: Add PyAPI_FUNC() to _PyDict_SizeOf() declaration. (#639) (diff) | |
| download | cpython-783d0c1a1c723733adcdf4249240246fc35a5bcb.tar.gz cpython-783d0c1a1c723733adcdf4249240246fc35a5bcb.tar.bz2 cpython-783d0c1a1c723733adcdf4249240246fc35a5bcb.zip | |
bpo-28667: Fix a compile warning on FreeBSD when compare with FD_SETSIZE. (#501)
FreeBSD is the only platforms with unsigned FD_SETSIZE.
Diffstat (limited to 'Modules/selectmodule.c')
| -rw-r--r-- | Modules/selectmodule.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/Modules/selectmodule.c b/Modules/selectmodule.c index 8bdf3359d48..6f71d585568 100644 --- a/Modules/selectmodule.c +++ b/Modules/selectmodule.c @@ -68,8 +68,8 @@ typedef struct { static void reap_obj(pylist fd2obj[FD_SETSIZE + 1]) { - int i; - for (i = 0; i < FD_SETSIZE + 1 && fd2obj[i].sentinel >= 0; i++) { + unsigned int i; + for (i = 0; i < (unsigned int)FD_SETSIZE + 1 && fd2obj[i].sentinel >= 0; i++) { Py_CLEAR(fd2obj[i].obj); } fd2obj[0].sentinel = -1; @@ -83,7 +83,7 @@ static int seq2set(PyObject *seq, fd_set *set, pylist fd2obj[FD_SETSIZE + 1]) { int max = -1; - int index = 0; + unsigned int index = 0; Py_ssize_t i; PyObject* fast_seq = NULL; PyObject* o = NULL; @@ -120,7 +120,7 @@ seq2set(PyObject *seq, fd_set *set, pylist fd2obj[FD_SETSIZE + 1]) FD_SET(v, set); /* add object and its file descriptor to the list */ - if (index >= FD_SETSIZE) { + if (index >= (unsigned int)FD_SETSIZE) { PyErr_SetString(PyExc_ValueError, "too many file descriptors in select()"); goto finally; |