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

Fixed documentation comment of H5A.get_type #173

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

myd7349
Copy link

@myd7349 myd7349 commented Apr 29, 2021

The documentation comment of H5A.get_type seems incorrect:

public extern static hsize_t get_storage_size(hid_t attr_id);
/// <summary>
/// Returns the amount of storage required for an attribute.
/// See https://www.hdfgroup.org/HDF5/doc/RM/RM_H5A.html#Annot-GetType
/// </summary>
/// <param name="attr_id">Identifier of an attribute.</param>
/// <returns>Returns a datatype identifier if successful; otherwise
/// returns a negative value.</returns>
[DllImport(Constants.DLLFileName, EntryPoint = "H5Aget_type",
CallingConvention = CallingConvention.Cdecl),
SuppressUnmanagedCodeSecurity, SecuritySafeCritical]
public extern static hid_t get_type(hid_t attr_id);

Fix it according to: https://portal.hdfgroup.org/display/HDF5/H5A_GET_TYPE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant