WIP: avoid double inclusion of abi.h

Author: Joe-Downs

ompi/mpi/bindings/c_header.py

@@ -215,11 +215,6 @@ def output_constant(const, use_enum: bool, mangle_name: bool):
          output.append(f"{binding[0]} P{' '.join(binding[1:])};\n")
 
 # ================================ Final Output ================================
-output.append("""#ifndef _ABI_INTERNAL_
-#define _ABI_INTERNAL_
-#include "stddef.h"
-#include "stdint.h"
-""")
 output.append("#endif /* _ABI_INTERNAL_ */")
 
 for i, line in enumerate(output):

ompi/mpi/c/abi.h.in

@@ -6,17 +6,22 @@
  * $HEADER$
  */
 
+#ifndef _ABI_INTERNAL_
+#define _ABI_INTERNAL_
+
+/* TODO: figure out which includes we need, change to <> instead of "" */
 #include "stddef.h"
 #include "stdint.h"
 
-//#include <stdint.h>
+/* #include <stdint.h> */
 
 #define MPI_VERSION    5
 #define MPI_SUBVERSION 0
 
 #define MPI_ABI_VERSION    1
 #define MPI_ABI_SUBVERSION 0
 
+/* TODO: add comment why this is hardcoded to avoid extra generation logic */
 typedef struct MPI_ABI_Comm * MPI_Comm;
 typedef struct MPI_ABI_Datatype * MPI_Datatype;
 typedef struct MPI_ABI_Errhandler * MPI_Errhandler;