Skip to content

559 support external declarations#561

Open
Joerg Henrichs (hiker) wants to merge 7 commits intomainfrom
559_support_external_declarations
Open

559 support external declarations#561
Joerg Henrichs (hiker) wants to merge 7 commits intomainfrom
559_support_external_declarations

Conversation

@hiker
Copy link
Copy Markdown
Collaborator

Adds support for external declarations.

@hiker
Copy link
Copy Markdown
Collaborator Author

Joerg Henrichs (hiker) commented Apr 24, 2026

This is blocked by #558 (it modifies the same file, so will likely cause conflicts otherwise)

@hiker
Copy link
Copy Markdown
Collaborator Author

I've also added support for external attributes (which is from the Fortran grammar point of view different from an external statement), and fixed an issue that subroutines in interfaces where not recorded as dependencies - we clearly lacked a test case there, which I have added.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Adds to a previous change. Seems to make sense.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I notice that this branch is not at head of trunk. It's probably best if the developer performs the merge.

@hiker
Copy link
Copy Markdown
Collaborator Author

I notice that this branch is not at head of trunk. It's probably best if the developer performs the merge.

Yes, it was blocked by 558 (do you prefer me to mark them as 'draft' instead?). I've merged it all together, should be good to go now

@hiker Joerg Henrichs (hiker) added Ready for review Indicating that a PR is ready to be reviewed. and removed Blocked Blocked by another issue labels May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready for review Indicating that a PR is ready to be reviewed.

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants