Merge pull request matplotlib#2991 from jkseppan/tidyup-ft2font

Silence some compiler warnings related to ft2font

Author: mdboom

src/file_compat.h

@@ -192,15 +192,15 @@ mpl_PyFile_OpenFile(PyObject *filename, const char *mode)
     if (open == NULL) {
         return NULL;
     }
-    return PyObject_CallFunction(open, "Os", filename, mode);
+    return PyObject_CallFunction(open, (char*)"Os", filename, mode);
 }
 
 static NPY_INLINE int
 mpl_PyFile_CloseFile(PyObject *file)
 {
     PyObject *ret;
 
-    ret = PyObject_CallMethod(file, "close", NULL);
+    ret = PyObject_CallMethod(file, (char*)"close", NULL);
     if (ret == NULL) {
         return -1;
     }

src/ft2font.cpp

@@ -193,7 +193,10 @@ FT2Image::py_write_bitmap(const Py::Tuple & args)
 
     write_bitmap(fh);
 
-    mpl_PyFile_DupClose(py_file, fh, offset);
+    if (mpl_PyFile_DupClose(py_file, fh, offset))
+    {
+        throw Py::Exception();
+    }
     mpl_PyFile_CloseFile(py_file);
     Py_DECREF(py_file);
 
@@ -2143,7 +2146,10 @@ static void close_file_callback(FT_Stream stream)
 {
     py_file_def *def = (py_file_def *)stream->descriptor.pointer;
 
-    mpl_PyFile_DupClose(def->py_file, def->fp, def->offset);
+    if (mpl_PyFile_DupClose(def->py_file, def->fp, def->offset))
+    {
+        throw Py::Exception();
+    }
 
     if (def->close_file) {
         mpl_PyFile_CloseFile(def->py_file);
@@ -2165,7 +2171,7 @@ FT2Font::make_open_args(PyObject *py_file_arg, FT_Open_Args *open_args)
     py_file_def *stream_info = NULL;
     long file_size;
     FT_Byte *new_memory;
-    mpl_off_t offset;
+    mpl_off_t offset = 0;
 
     int result = -1;
 
@@ -2208,7 +2214,7 @@ FT2Font::make_open_args(PyObject *py_file_arg, FT_Open_Args *open_args)
         open_args->stream = &stream;
     } else {
         if (PyObject_HasAttrString(py_file_arg, "read") &&
-            (data = PyObject_CallMethod(py_file_arg, "read", ""))) {
+            (data = PyObject_CallMethod(py_file_arg, (char*)"read", (char*)""))) {
             if (PyBytes_AsStringAndSize(data, &data_ptr, &data_len)) {
                 goto exit;
             }