diff options
author | Dana Robinson <43805+derobins@users.noreply.github.com> | 2023-05-30 15:33:52 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-05-30 15:33:52 (GMT) |
commit | 81bc34ac4c3f42532edee32095651f6bcd5e55a2 (patch) | |
tree | 31c71ea5ac918620c21133d77a44cae70f1ef289 /src/H5Fefc.c | |
parent | 962593fb0aaabe8cc8d05e162e4bd2745e164f0b (diff) | |
download | hdf5-81bc34ac4c3f42532edee32095651f6bcd5e55a2.zip hdf5-81bc34ac4c3f42532edee32095651f6bcd5e55a2.tar.gz hdf5-81bc34ac4c3f42532edee32095651f6bcd5e55a2.tar.bz2 |
Only pass efc pointer to H5F__efc_open (#3026)
The H5F__efc_open() API call only needs the efc pointer, so there is
no need to pass the entire file struct in.
Diffstat (limited to 'src/H5Fefc.c')
-rw-r--r-- | src/H5Fefc.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/src/H5Fefc.c b/src/H5Fefc.c index ea48054..e7de395 100644 --- a/src/H5Fefc.c +++ b/src/H5Fefc.c @@ -137,9 +137,8 @@ done: *------------------------------------------------------------------------- */ H5F_t * -H5F__efc_open(H5F_t *parent, const char *name, unsigned flags, hid_t fcpl_id, hid_t fapl_id) +H5F__efc_open(H5F_efc_t *efc, const char *name, unsigned flags, hid_t fcpl_id, hid_t fapl_id) { - H5F_efc_t *efc = NULL; /* External file cache for parent file */ H5F_efc_ent_t *ent = NULL; /* Entry for target file in efc */ hbool_t open_file = FALSE; /* Whether ent->file needs to be closed in case of error */ H5P_genplist_t *plist; /* Property list pointer for FAPL */ @@ -149,8 +148,6 @@ H5F__efc_open(H5F_t *parent, const char *name, unsigned flags, hid_t fcpl_id, hi FUNC_ENTER_PACKAGE /* Sanity checks */ - HDassert(parent); - HDassert(parent->shared); HDassert(name); /* Get the VOL info from the fapl */ @@ -165,9 +162,6 @@ H5F__efc_open(H5F_t *parent, const char *name, unsigned flags, hid_t fcpl_id, hi if (H5CX_set_vol_connector_prop(&connector_prop) < 0) HGOTO_ERROR(H5E_FILE, H5E_CANTSET, NULL, "can't set VOL connector info in API context") - /* Get external file cache */ - efc = parent->shared->efc; - /* Check if the EFC exists. If it does not, just call H5F_open(). We * support this so clients do not have to make 2 different calls depending * on the state of the efc. */ |