Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add bug note to H5Dget_space_status documentation (#2788) #2837

Merged
merged 1 commit into from
Apr 30, 2023

Conversation

jhendersonHDF
Copy link
Collaborator

No description provided.

@jhendersonHDF jhendersonHDF added Merge Use this label when a PR is for a downstream merge Priority - 2. Medium ⏹ It would be nice to have this in the next release Component - Documentation Doxygen, markdown, etc. Type - Task Actions that don't fit into any other type category Branch - 1.10 labels Apr 27, 2023
src/H5Dpublic.h Outdated
@@ -423,6 +423,20 @@ H5_DLL hid_t H5Dget_space(hid_t dset_id);
* \details H5Dget_space_status() determines whether space has been allocated
* for the dataset \p dset_id.
*
* \note \Bold{BUG:} Prior to the HDF5 1.13.1, 1.12.2 and 1.10.9 releases,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought we treat 1.13.x as if they never exist...

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's true, although changing this to 1.14.0 would be slightly inaccurate. I've also seen 1.13 releases being used in the wild, which complicates documenting this a bit as well.

@jhendersonHDF jhendersonHDF force-pushed the H5Dget_space_status_1_10 branch from 4dcd962 to 39530c9 Compare April 27, 2023 19:27
@derobins derobins merged commit 556bc50 into HDFGroup:hdf5_1_10 Apr 30, 2023
@jhendersonHDF jhendersonHDF deleted the H5Dget_space_status_1_10 branch May 5, 2023 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Documentation Doxygen, markdown, etc. Merge Use this label when a PR is for a downstream merge Priority - 2. Medium ⏹ It would be nice to have this in the next release Type - Task Actions that don't fit into any other type category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants