Skip to content

Fix parser failing on field types that wrap across lines#3535

Merged
oldergod merged 2 commits intosquare:masterfrom
jhansche:fix/3281
Mar 11, 2026
Merged

Fix parser failing on field types that wrap across lines#3535
oldergod merged 2 commits intosquare:masterfrom
jhansche:fix/3281

Conversation

@jhansche
Copy link
Contributor

When a qualified type name in a field declaration was split across lines (e.g. with a leading dot on the continuation line), the parser would stop reading the type at the newline and misinterpret the rest. Now readDataType() peeks ahead after reading a word and continues consuming dot-separated segments across line breaks.

Fixes #3281

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

When a qualified type name in a field declaration was split across lines
(e.g. with a leading dot on the continuation line), the parser would stop
reading the type at the newline and misinterpret the rest. Now
readDataType() peeks ahead after reading a word and continues consuming
dot-separated segments across line breaks.

Fixes square#3281

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jhansche
Copy link
Contributor Author

Sorry I missed it earlier: CLA has been signed.

@oldergod oldergod enabled auto-merge March 11, 2026 10:02
@oldergod
Copy link
Member

Thank you

@oldergod oldergod merged commit 47ff61c into square:master Mar 11, 2026
10 checks passed
@oldergod oldergod changed the title Fix parser failing on field types that wrap across lines [fixes #3281] Fix parser failing on field types that wrap across lines Mar 11, 2026
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.

Allow whitespace and comments in identifiers when parsing

2 participants