Skip to content

feat: warning if file setup parameters are ignored#879

Open
jodavies wants to merge 1 commit into
form-dev:masterfrom
jodavies:file-late-setup
Open

feat: warning if file setup parameters are ignored#879
jodavies wants to merge 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

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