From af7046bc28f72e2fe03727a8a4e56bd29c195e4e Mon Sep 17 00:00:00 2001
From: David Gardner <dagardner@nvidia.com>
Date: Mon, 1 Apr 2024 11:09:19 -0700
Subject: [PATCH] IWYU fixes

---
 python/mrc/_pymrc/include/pymrc/types.hpp       |  2 +-
 python/mrc/_pymrc/include/pymrc/utils.hpp       |  1 -
 python/mrc/_pymrc/src/utilities/json_values.cpp | 13 +++++++------
 python/mrc/_pymrc/src/utils.cpp                 |  1 +
 4 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/python/mrc/_pymrc/include/pymrc/types.hpp b/python/mrc/_pymrc/include/pymrc/types.hpp
index 2aea06944..5446ec28a 100644
--- a/python/mrc/_pymrc/include/pymrc/types.hpp
+++ b/python/mrc/_pymrc/include/pymrc/types.hpp
@@ -24,7 +24,7 @@
 #include <nlohmann/json_fwd.hpp>
 #include <rxcpp/rx.hpp>
 
-#include <functional>
+#include <functional>  // for function
 #include <map>
 #include <string>
 
diff --git a/python/mrc/_pymrc/include/pymrc/utils.hpp b/python/mrc/_pymrc/include/pymrc/utils.hpp
index 3d1cf7b9c..714605e6a 100644
--- a/python/mrc/_pymrc/include/pymrc/utils.hpp
+++ b/python/mrc/_pymrc/include/pymrc/utils.hpp
@@ -24,7 +24,6 @@
 #include <pybind11/pytypes.h>
 #include <sys/types.h>
 
-#include <functional>  // for function
 #include <string>
 #include <typeinfo>
 
diff --git a/python/mrc/_pymrc/src/utilities/json_values.cpp b/python/mrc/_pymrc/src/utilities/json_values.cpp
index 03417feb8..a75576fc6 100644
--- a/python/mrc/_pymrc/src/utilities/json_values.cpp
+++ b/python/mrc/_pymrc/src/utilities/json_values.cpp
@@ -26,12 +26,13 @@
 #include <glog/logging.h>
 #include <pybind11/cast.h>
 
-#include <iterator>   // for next
-#include <map>        // for map
-#include <sstream>    // for operator<< & stringstream
-#include <stdexcept>  // for runtime_error
-#include <utility>    // for move
-#include <vector>     // for vector
+#include <functional>  // for function
+#include <iterator>    // for next
+#include <map>         // for map
+#include <sstream>     // for operator<< & stringstream
+#include <stdexcept>   // for runtime_error
+#include <utility>     // for move
+#include <vector>      // for vector
 
 // We already have <boost/algorithm/string.hpp> included we don't need these others, it is also the only public header
 // with a definition for boost::is_any_of, so even if we replaced string.hpp with these others we would still need to
diff --git a/python/mrc/_pymrc/src/utils.cpp b/python/mrc/_pymrc/src/utils.cpp
index 3477f6c8f..22379b594 100644
--- a/python/mrc/_pymrc/src/utils.cpp
+++ b/python/mrc/_pymrc/src/utils.cpp
@@ -28,6 +28,7 @@
 #include <pyerrors.h>
 #include <warnings.h>
 
+#include <functional>  // for function
 #include <sstream>
 #include <string>
 #include <utility>