-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v: add map update-init syntax #20561
Conversation
@spytheman are you able to retry the failing test? It seems unrelated? |
restarted. |
has_update_expr bool // has `...a` | ||
update_expr Expr // `a` in `...a` | ||
update_expr_pos token.Pos | ||
update_expr_comments []Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cmd/tools/vast/vast.v:1645
also needs updating, when adding new fields, that can carry meaningful information about the source. v ast -p file.v
is very useful, for understanding the AST structure of a file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent work.
Add support for map update-init syntax to v:
which is functionally equivalent to:
except that it yields an expression which does not have to be mutable.
This PR updates:
new_map_update_init()
to builtin/map.v to support cgen