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

BUG: f2py: better handle filtering of public/private subroutines #27304

Merged
merged 2 commits into from
Aug 28, 2024

Conversation

charris
Copy link
Member

@charris charris commented Aug 28, 2024

Backport of #27049.

This should fix gh-26920 and add a corresponding test case.

I am unsure about some formalities and would appreciate some feedback:

  • I am not sure if I put the test in the correct location
  • Technically, this fixes another underlying bug: Handling of f90 modules that have no public variables or subroutines. I'm not sure if this warrants a separate test, but please let me know and I will reformat accordingly.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Don't mistake public/private declarations of F90 subroutines for variables
when the corresponding subroutines are filtered by use of only:.
Also, handle modules with no public variables or subroutines, caused by
the filtering.

Closes numpygh-26920.
@charris charris added this to the 2.1.1 release milestone Aug 28, 2024
@charris charris merged commit ca7f5c1 into numpy:maintenance/2.1.x Aug 28, 2024
65 checks passed
@charris charris deleted the backport-27049 branch August 28, 2024 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants