Skip to content

BUG: Fix f2py bugs when wrapping F90 subroutines.#18184

Merged
charris merged 8 commits intonumpy:masterfrom
pearu:17797
Jan 19, 2021
Merged

BUG: Fix f2py bugs when wrapping F90 subroutines.#18184
charris merged 8 commits intonumpy:masterfrom
pearu:17797

Conversation

@pearu
Copy link
Copy Markdown
Contributor

@pearu pearu commented Jan 18, 2021

This PR fixes the following bugs:

  • Fortran sources in fixed format were incorrectly detected as free format sources
  • f2py specific modules (with names containing __user__) must not be used in Fortran sources to be compiled
  • Subroutines/functions using assumed shape array inputs were incorrectly treated as F77 codes. Closes f2py: assumed shape array arguments break callback functionality #17797
  • Avoid splitting long comment lines

The PR supersedes #17800

@pearu pearu self-assigned this Jan 18, 2021
@pearu pearu marked this pull request as ready for review January 18, 2021 19:29
@charris
Copy link
Copy Markdown
Member

charris commented Jan 19, 2021

LGTM, just a couple of style suggestions.

@pearu pearu requested a review from charris January 19, 2021 09:03
@pearu pearu requested a review from charris January 19, 2021 19:36
@charris charris merged commit efaf210 into numpy:master Jan 19, 2021
@charris
Copy link
Copy Markdown
Member

charris commented Jan 19, 2021

Thanks Pearu.

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.

f2py: assumed shape array arguments break callback functionality

3 participants