HDFGroup / vol-rest

HDF5 REST VOL Connector
Other
5 stars 8 forks source link

Implement 'by index' functions #37

Closed mattjala closed 1 year ago

mattjala commented 1 year ago

Implemented support for H5Gget_info_by_name, H5Oget_info_by_idx, H5Aget_info_by_idx, H5Aget_name_by_idx, and H5Aopen_by_idx (#34)

Updated test_rest_vol.c to stop expecting some of those functions to fail.

Patched a memory error in H5Gget_info_by_name, and cleaned up some things that were causing warnings on build.

jhendersonHDF commented 1 year ago

There are a couple remaining minor cleanup tasks, but I think in the interest of getting this merged the changes for H5Ovisit should be split into a separate PR, while the changes for H5Oget_info_by_name/H5Oget_info_by_idx should stay in this PR. The changes for H5Ovisit are fairly extensive and will take some time to review.