BLD,BUG: Remove Python.h includes immediately preceeding pybind11/pybind11.h includes

DWesl · web-flow · commit 66149edad6dc · 2024-03-06T10:29:31.000-05:00
pybind11 seems decent about including Python.h before any system headers, once you chase three layers of includes in to see that.
diff --git a/src/_image_wrapper.cpp b/src/_image_wrapper.cpp
@@ -1,4 +1,3 @@
-#include <Python.h>
 #include <pybind11/pybind11.h>
 #include <pybind11/numpy.h>
 
diff --git a/src/_path.h b/src/_path.h
@@ -3,7 +3,6 @@
 #ifndef MPL_PATH_H
 #define MPL_PATH_H
 
-#include <Python.h>
 #include <limits>
 #include <math.h>
 #include <vector>
diff --git a/src/_qhull_wrapper.cpp b/src/_qhull_wrapper.cpp
@@ -5,7 +5,6 @@
  * triangulation, construct an instance of the matplotlib.tri.Triangulation
  * class without specifying a triangles array.
  */
-#include <Python.h>
 #include <pybind11/pybind11.h>
 #include <pybind11/numpy.h>
 
diff --git a/src/_ttconv.cpp b/src/_ttconv.cpp
@@ -5,10 +5,8 @@
 
   Python wrapper for TrueType conversion library in ../ttconv.
  */
-#include <Python.h>
-#include "mplutils.h"
-
 #include <pybind11/pybind11.h>
+#include "mplutils.h"
 #include "pprdrv.h"
 #include <vector>
 
diff --git a/src/py_converters_11.h b/src/py_converters_11.h
@@ -3,7 +3,6 @@
 
 // pybind11 equivalent of py_converters.h
 
-#include <Python.h>
 #include <pybind11/pybind11.h>
 #include <pybind11/numpy.h>
 
diff --git a/src/tri/_tri.h b/src/tri/_tri.h
@@ -63,7 +63,6 @@
 #ifndef MPL_TRI_H
 #define MPL_TRI_H
 
-#include <Python.h>
 #include <pybind11/pybind11.h>
 #include <pybind11/numpy.h>
 
