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

H5FD__init_package totally unused (it seems to me, and according to clang) #349

Closed
seanm opened this issue Feb 17, 2021 · 4 comments
Closed
Assignees

Comments

@seanm
Copy link
Contributor

seanm commented Feb 17, 2021

Many files have a function H5FD__init_package, usually static. It appears totally unused. Clang warns:

warning: unused function 'H5FD__init_package' [-Wunused-function]

Am I blind or is it really unused?

@qkoziol
Copy link
Contributor

qkoziol commented Feb 19, 2021

It's composed as a result of the H5_PACKAGE_INIT_FUNC "glue" macro, from the 'H5_MY_PKG' macro in src/H5FDmodule.h. It's not a great fit in the way it's currently used, but would be some work to refactor to eliminate. :-/

@jhendersonHDF jhendersonHDF self-assigned this Jun 3, 2021
@jhendersonHDF jhendersonHDF linked a pull request Jun 3, 2021 that will close this issue
@jhendersonHDF jhendersonHDF removed a link to a pull request Jun 3, 2021
@qkoziol
Copy link
Contributor

qkoziol commented Jun 3, 2021

This is used in the H5FDmodule.h header's macros

@jhendersonHDF
Copy link
Collaborator

This is used in the H5FDmodule.h header's macros

I initially thought this issue might be related because the previously-linked PR does clean up some warnings related to unused H5FD package variables and undefined H5FD routines, but it seems this is separate from that.

@derobins
Copy link
Member

derobins commented Jul 4, 2022

The reworked VFD initialization routines do not have this problem. These are in develop (1.13.x, eventually 1.14.0). They will probably not be merged down to 1.12 and earlier.

@derobins derobins closed this as completed Jul 4, 2022
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

No branches or pull requests

4 participants