Skip to content

CI check: 484_format_missing_comma (do not merge) - #2

Closed
DisciplinedSoftware wants to merge 1 commit into
masterfrom
484_format_missing_comma
Closed

CI check: 484_format_missing_comma (do not merge)#2
DisciplinedSoftware wants to merge 1 commit into
masterfrom
484_format_missing_comma

Conversation

@DisciplinedSoftware

Copy link
Copy Markdown
Owner

Draft PR to run CI on the fork while upstream stfc#526 awaits first-contributor workflow approval.

Many compilers (gfortran, ifort, ifx) accept a missing comma between
a character-string edit descriptor and a neighbouring format item,
e.g. FORMAT('a' 1x,'b'), FORMAT(15x'a') or FORMAT('a' 'b').

Following the extension convention, 'format-missing-comma' is added
to the EXTENSIONS list in utils.py and the relaxation is implemented
in Format_Item_C1002, whose match logic is split into the standard
C1002 pass and an extension fallback. Found while parsing MODFLOW-2005
(gwf2swr7.f, gwf2swi27.fpp, gwf2lak7.f, gwf2sfr7.f); also covers the
example in issue stfc#484.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@DisciplinedSoftware

Copy link
Copy Markdown
Owner Author

CI green (fparser unit tests: success). Closing — this was only a CI vehicle for upstream stfc#526.

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