json: fix option sumtype handling #20186
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix #20181
🤖[deprecated] Generated by Copilot at f83790b
Improve and fix
json_no_inline_sumtypes
flag for sumtypes. Add support for option sumtypes and struct variants invlib/v/gen/c/json.v
.🤖[deprecated] Generated by Copilot at f83790b
json_no_inline_sumtypes
flag by removing unnecessaryelse
branch (link)var_data
variable instead ofval
prefix for encoding sumtype variants that are time.Time or other structs to make code more consistent and avoid errors (link)json_no_inline_sumtypes
flag by wrapping variant value in_option_ok
call (link)