Skip to content

parser: smart detect or {} inside loop when using c <-val #25435

Merged
spytheman merged 6 commits into
vlang:masterfrom
Delta456:for_channel_or_block
Oct 27, 2025
Merged

parser: smart detect or {} inside loop when using c <-val #25435
spytheman merged 6 commits into
vlang:masterfrom
Delta456:for_channel_or_block

Conversation

@Delta456

@Delta456 Delta456 commented Oct 2, 2025

Copy link
Copy Markdown
Member

Fixes #24550

@huly-for-github

Copy link
Copy Markdown

Connected to Huly®: V_0.6-26119

@Delta456 Delta456 marked this pull request as draft October 3, 2025 09:04
@medvednikov medvednikov marked this pull request as ready for review October 16, 2025 15:27
@medvednikov

Copy link
Copy Markdown
Member

@Delta456 why draft still?

@Delta456

Copy link
Copy Markdown
Member Author

@Delta456 why draft still?

VLS used to fail, as my fix was messing up code generation. Now the CI seem to pass? That is strange.

@spytheman

Copy link
Copy Markdown
Contributor

vlang/vls had recent changes, perhaps they served as a workaround

@spytheman

Copy link
Copy Markdown
Contributor

I'll rebase the changes here over our current master, to check if everything is ok, and if it is, I'll merge it.

@spytheman spytheman force-pushed the for_channel_or_block branch from b0fe87f to 1768552 Compare October 17, 2025 16:30
@Delta456

Copy link
Copy Markdown
Member Author

The CI still fails https://github.com/vlang/v/actions/runs/18598848534/job/53032048538?pr=25435. I haven't got time to fix this issue lately. I would appreciate if someone fixes it.

Comment thread vlib/v/checker/checker.v Outdated
@Delta456 Delta456 force-pushed the for_channel_or_block branch from 1768552 to 5d5b812 Compare October 26, 2025 08:20
@Delta456 Delta456 force-pushed the for_channel_or_block branch from 5d5b812 to b89000f Compare October 26, 2025 08:20
@Delta456

Copy link
Copy Markdown
Member Author

Thanks @StunxFS for the fix 🫡

@Delta456 Delta456 requested a review from StunxFS October 26, 2025 10:49

@StunxFS StunxFS left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work bro 🫡

@spytheman spytheman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work.
Thank you @Delta456.
Thank you @StunxFS.

@spytheman spytheman merged commit 13efcad into vlang:master Oct 27, 2025
84 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.

channel or { } for closing detection confused with method option/result or

4 participants