diff options
| author | Dana Robinson <43805+derobins@users.noreply.github.com> | 2024-03-09 07:17:13 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-03-09 07:17:13 (GMT) |
| commit | 3d09a7d5f6dd0f602a8f343809384ed00cd13f5c (patch) | |
| tree | 00ea6998f2c44ae9d0e6460bc617b0639abc2734 /src/H5Bprivate.h | |
| parent | d8af09dd8f064c97c4120b2a5d1cfd1103526d07 (diff) | |
| download | hdf5-3d09a7d5f6dd0f602a8f343809384ed00cd13f5c.zip hdf5-3d09a7d5f6dd0f602a8f343809384ed00cd13f5c.tar.gz hdf5-3d09a7d5f6dd0f602a8f343809384ed00cd13f5c.tar.bz2 | |
Remove H5B debug checks (#4089)
The H5B (version 1 B-tree) package would add some computationally
expensive integrity checks when H5B_DEBUG was defined. Due to their
negative effects on performance, this option was rarely turned on,
making the H5B__assert() check function stale, if not dead, code.
This change:
* Builds H5B__assert() when NDEBUG is not defined (the function
relies on assert()) so it gets compiled more often.
* Removes some printf debugging statements in the B-tree code
* Removes all H5B "extra debug" checks that are leftover from
past debugging sessions. Maintainers can add H5B__assert()
selectively to perform integrity checks when debugging.
* Removes the HDF5_ENABLE_DEBUG_H5B CMake option
H5B_DEBUG now has no effect
Diffstat (limited to 'src/H5Bprivate.h')
| -rw-r--r-- | src/H5Bprivate.h | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/src/H5Bprivate.h b/src/H5Bprivate.h index a1d84ef..f93fa9c 100644 --- a/src/H5Bprivate.h +++ b/src/H5Bprivate.h @@ -31,16 +31,6 @@ /* Library Private Macros */ /**************************/ -/* - * Feature: Define this constant if you want to check B-tree consistency - * after each B-tree operation. Note that this slows down the - * library considerably! Debugging the B-tree depends on assert() - * being enabled. - */ -#ifdef NDEBUG -#undef H5B_DEBUG -#endif - /****************************/ /* Library Private Typedefs */ /****************************/ |