Avoid calling into H5open if the library is already initialized

Signed-off-by: Quincey Koziol <[email protected]>
qkoziol · Oct 3, 2024 · d76c617 · d76c617
1 parent cdb637b
commit d76c617
Show file tree

Hide file tree

Showing 22 changed files with 34 additions and 23 deletions.
diff --git a/src/H5Epublic.h b/src/H5Epublic.h
@@ -52,7 +52,7 @@ typedef struct H5E_error2_t {
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDcore.h b/src/H5FDcore.h
@@ -22,7 +22,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDdirect.h b/src/H5FDdirect.h
@@ -24,7 +24,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDfamily.h b/src/H5FDfamily.h
@@ -22,7 +22,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDhdfs.h b/src/H5FDhdfs.h
@@ -26,7 +26,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDlog.h b/src/H5FDlog.h
@@ -23,7 +23,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDmirror.h b/src/H5FDmirror.h
@@ -25,7 +25,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDmpio.h b/src/H5FDmpio.h
@@ -24,7 +24,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDmulti.h b/src/H5FDmulti.h
@@ -23,7 +23,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDonion.h b/src/H5FDonion.h
@@ -22,7 +22,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDros3.h b/src/H5FDros3.h
@@ -24,7 +24,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDsec2.h b/src/H5FDsec2.h
@@ -23,7 +23,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDsplitter.h b/src/H5FDsplitter.h
@@ -23,7 +23,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDstdio.h b/src/H5FDstdio.h
@@ -23,7 +23,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDsubfiling/H5FDioc.h b/src/H5FDsubfiling/H5FDioc.h
@@ -28,7 +28,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDsubfiling/H5FDsubfiling.h b/src/H5FDsubfiling/H5FDsubfiling.h
@@ -22,7 +22,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5FDwindows.h b/src/H5FDwindows.h
@@ -26,7 +26,7 @@
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5Opublic.h b/src/H5Opublic.h
@@ -1681,6 +1681,14 @@ H5_DLL herr_t H5Otoken_from_str(hid_t loc_id, const char *token_str, H5O_token_t
 #endif
 /// \endcond
 
+/* When this header is included from a private header, don't make calls to H5open() */
+#undef H5OPEN
+#ifndef H5private_H
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
+#else /* H5private_H */
+#define H5OPEN
+#endif /* H5private_H */
+
 /* The canonical 'undefined' token value */
 #define H5O_TOKEN_UNDEF (H5OPEN H5O_TOKEN_UNDEF_g)
 H5_DLLVAR const H5O_token_t H5O_TOKEN_UNDEF_g;

diff --git a/src/H5Ppublic.h b/src/H5Ppublic.h
@@ -37,7 +37,7 @@
 /* When this header is included from a private HDF5 header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5Tpublic.h b/src/H5Tpublic.h
@@ -242,7 +242,7 @@ typedef H5T_conv_ret_t (*H5T_conv_except_func_t)(H5T_conv_except_t except_type,
 /* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
 #ifndef H5private_H
-#define H5OPEN H5open(),
+#define H5OPEN ((!H5_libinit_g && !H5_libterm_g) ? H5open() : 0),
 #else /* H5private_H */
 #define H5OPEN
 #endif /* H5private_H */

diff --git a/src/H5private.h b/src/H5private.h
@@ -1207,10 +1207,6 @@ extern char H5_lib_vers_info_g[];
 
 #endif /* H5_HAVE_THREADSAFE */
 
-/* Library init / term status (global) */
-extern bool H5_libinit_g; /* Has the library been initialized? */
-extern bool H5_libterm_g; /* Is the library being shutdown? */
-
 /* Macros for accessing the global variables */
 #define H5_INIT_GLOBAL (H5_libinit_g)
 #define H5_TERM_GLOBAL (H5_libterm_g)

diff --git a/src/H5public.h b/src/H5public.h
@@ -434,6 +434,13 @@ typedef void (*H5_atclose_func_t)(void *ctx);
 extern "C" {
 #endif
 
+/** @private
+ *
+ * \brief Library init / term status (global)
+ */
+H5_DLLVAR bool H5_libinit_g; /* Has the library been initialized? */
+H5_DLLVAR bool H5_libterm_g; /* Is the library being shutdown? */
+
 /* Functions in H5.c */
 /**
  * \ingroup H5