commit 28f468cb19e3097079b7ce7850e6048de99022fa
author Victor Stinner <vstinner@redhat.com> Thu Nov 22 13:21:43 2018 +0100
committer GitHub <noreply@github.com> Thu Nov 22 13:21:43 2018 +0100
tree 0bda352ec8b1777c3790882e315c97eb939119de
parent cdbcb773f5db24e23fa90e644ec620d54bd08127

bpo-9566: Fix compiler warnings in pyexpat.c (GH-10654)

Explicit cast a pointer difference (intptr_t) to int to fix
two warnings on 64-bit Windows:

    Modules\pyexpat.c(1181): warning C4244: 'initializing':
    conversion from '__int64' to 'int', possible  loss of data

    Modules\pyexpat.c(1192): warning C4244: 'initializing':
    conversion from '__int64' to 'int', possible  loss of data

Modules/pyexpat.c

diff --git a/Modules/pyexpat.c b/Modules/pyexpat.c
index 10d5aed..9384081 100644
--- a/Modules/pyexpat.c
+++ b/Modules/pyexpat.c
@@ -1178,7 +1178,8 @@
 static PyObject *
 xmlparse_handler_getter(xmlparseobject *self, struct HandlerInfo *hi)
 {
-    int handlernum = hi - handler_info;
+    assert((hi - handler_info) < (Py_ssize_t)Py_ARRAY_LENGTH(handler_info));
+    int handlernum = (int)(hi - handler_info);
     PyObject *result = self->handlers[handlernum];
     if (result == NULL)
         result = Py_None;
@@ -1189,7 +1190,8 @@
 static int
 xmlparse_handler_setter(xmlparseobject *self, PyObject *v, struct HandlerInfo *hi)
 {
-    int handlernum = hi - handler_info;
+    assert((hi - handler_info) < (Py_ssize_t)Py_ARRAY_LENGTH(handler_info));
+    int handlernum = (int)(hi - handler_info);
     if (v == NULL) {
         PyErr_SetString(PyExc_RuntimeError, "Cannot delete attribute");
         return -1;