BUG: Fix the returned value for PyErr_SetFromWindowsErr

DWesl · web-flow · commit 89249134fef3 · 2022-05-18T12:40:16.000-04:00
I didn't notice that PyErr_SetString returned void.
This should return something normal.
diff --git a/src/_tkagg.cpp b/src/_tkagg.cpp
@@ -25,7 +25,8 @@
  */
 #define WIN32_DLL
 static inline PyObject *PyErr_SetFromWindowsErr(int ierr) {
-    return PyErr_SetString(PyExc_OSError, "Call to EnumProcessModules failed");
+    PyErr_SetString(PyExc_OSError, "Call to EnumProcessModules failed");
+    return NULL;
 }
 #endif
 

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,8 @@`
`25`	`25`	`*/`
`26`	`26`	`#define WIN32_DLL`
`27`	`27`	`static inline PyObject *PyErr_SetFromWindowsErr(int ierr) {`
`28`		`- return PyErr_SetString(PyExc_OSError, "Call to EnumProcessModules failed");`
	`28`	`+ PyErr_SetString(PyExc_OSError, "Call to EnumProcessModules failed");`
	`29`	`+ return NULL;`
`29`	`30`	`}`
`30`	`31`	`#endif`
`31`	`32`