diff options
| author | Dana Robinson <43805+derobins@users.noreply.github.com> | 2021-05-11 16:05:03 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-05-11 16:05:03 (GMT) |
| commit | 7d140b97fbd78d8071780e38e3e4287f6711f7f6 (patch) | |
| tree | 919d01a7fea79441cc40b38ea616b8c51e200f4d /src/H5Gent.c | |
| parent | 9023e98940d959aa974e655cc383fab0b0ed663c (diff) | |
| download | hdf5-7d140b97fbd78d8071780e38e3e4287f6711f7f6.zip hdf5-7d140b97fbd78d8071780e38e3e4287f6711f7f6.tar.gz hdf5-7d140b97fbd78d8071780e38e3e4287f6711f7f6.tar.bz2 | |
Purges UFAIL from the library (#637)
* Committing clang-format changes
* Purges UFAIL from the library
* H5HL_insert change requested in PR
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Diffstat (limited to 'src/H5Gent.c')
| -rw-r--r-- | src/H5Gent.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/src/H5Gent.c b/src/H5Gent.c index b26e308..2f1a2b4 100644 --- a/src/H5Gent.c +++ b/src/H5Gent.c @@ -384,11 +384,8 @@ H5G__ent_convert(H5F_t *f, H5HL_t *heap, const char *name, const H5O_link_t *lnk /* Reset the new entry */ H5G__ent_reset(ent); - /* - * Add the new name to the heap. - */ - name_offset = H5HL_insert(f, heap, HDstrlen(name) + 1, name); - if (0 == name_offset || UFAIL == name_offset) + /* Add the new name to the heap */ + if (H5HL_insert(f, heap, HDstrlen(name) + 1, name, &name_offset) < 0) HGOTO_ERROR(H5E_SYM, H5E_CANTINSERT, FAIL, "unable to insert symbol name into heap") ent->name_off = name_offset; @@ -476,14 +473,12 @@ H5G__ent_convert(H5F_t *f, H5HL_t *heap, const char *name, const H5O_link_t *lnk size_t lnk_offset; /* Offset to sym-link value */ /* Insert link value into local heap */ - if (UFAIL == - (lnk_offset = H5HL_insert(f, heap, HDstrlen(lnk->u.soft.name) + 1, lnk->u.soft.name))) + if (H5HL_insert(f, heap, HDstrlen(lnk->u.soft.name) + 1, lnk->u.soft.name, &lnk_offset) < 0) HGOTO_ERROR(H5E_SYM, H5E_CANTINIT, FAIL, "unable to write link value to local heap") ent->type = H5G_CACHED_SLINK; ent->cache.slink.lval_offset = lnk_offset; - } /* end case */ - break; + } break; case H5L_TYPE_ERROR: case H5L_TYPE_EXTERNAL: |