Skip to content

docs(dfns): fix reason for inline-array position rule - #393

Merged
wpbonelli merged 1 commit into
MODFLOW-ORG:developfrom
wpbonelli:inline-array-rule-reason
Oct 14, 2026
Merged

wpbonelli merged 1 commit into
MODFLOW-ORG:developfrom
wpbonelli:inline-array-rule-reason

Conversation

@wpbonelli

Copy link
Copy Markdown
Member

An inline array whose width varies by row (self-sizing, sibling- or lookup-sized) must be a record's last field. The old reason, that a reader couldn't tell where later fields start, is wrong for a sibling, which comes earlier in the row. The reason is that a row's layout should be fixed by explicit dimensions, except for its last field.

An inline array whose width varies by row (self-sizing, sibling- or
lookup-sized) must be a record's last field. The old reason, that a reader
couldn't tell where later fields start, is wrong for a sibling, which comes
earlier in the row. The reason is that a row's layout should be fixed by
the component's dimensions, except for its last field, so a row can be
split without evaluating any other field, row or block. A union's arm, and
so its width, is chosen by its own tokens, so a union may come anywhere
(e.g. utl-obs's id, followed by id2).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@wpbonelli wpbonelli added this to the 1.10.0 milestone Oct 14, 2026
@wpbonelli wpbonelli added bug Something isn't working documentation Improvements or additions to documentation labels Oct 14, 2026
@wpbonelli wpbonelli changed the title docs(dfns): give the real reason for the inline-array position rule docs(dfns): fix reason for inline-array position rule Oct 14, 2026
@wpbonelli
wpbonelli marked this pull request as ready for review October 14, 2026 23:35
@wpbonelli
wpbonelli merged commit 61638bd into MODFLOW-ORG:develop Oct 14, 2026
3 of 4 checks passed
@wpbonelli
wpbonelli deleted the inline-array-rule-reason branch October 14, 2026 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant