Skip to content

feat: warning if file setup parameters are ignored#879

Merged
jodavies merged 1 commit into
form-dev:masterfrom
jodavies:file-late-setup
Jul 21, 2026
Merged

feat: warning if file setup parameters are ignored#879
jodavies merged 1 commit into
form-dev:masterfrom
jodavies:file-late-setup

Conversation

@jodavies

Copy link
Copy Markdown
Collaborator

If file setup parameters (#: ...) are provided, but there are non-comment lines preceeding them, they are silently ignored. This is consistent with the manual, but easy to do by accident. Add a warning in this case.


This is not perfect, in that it won't catch things like

#: MaxTermSize 50K
#define TEST "10"
#: SmallSize 1G

but it will catch (probably) most instances of this.

@coveralls

coveralls commented Jul 16, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 63.202% (+0.02%) from 63.186% — jodavies:file-late-setup into form-dev:master

If file setup parameters (#: ...) are provided, but there are
non-comment lines preceeding them, they are silently ignored.
This is consistent with the manual, but easy to do by accident.
Add a warning in this case.
@jodavies

Copy link
Copy Markdown
Collaborator Author

Updated version, which also warns in cases like

#: MaxTermSize 50K
#define TEST "10"
#: SmallSize 1G

@jodavies
jodavies merged commit 4ffb6a2 into form-dev:master Jul 21, 2026
106 checks passed
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.

2 participants