Skip to content

markused: fix []T param as used#25454

Merged
spytheman merged 8 commits into
vlang:masterfrom
felipensp:markused_generic_array_param
Oct 12, 2025
Merged

markused: fix []T param as used#25454
spytheman merged 8 commits into
vlang:masterfrom
felipensp:markused_generic_array_param

Conversation

@felipensp

Copy link
Copy Markdown
Member

Fix #25383

@huly-for-github

Copy link
Copy Markdown

Connected to Huly®: V_0.6-26138

@spytheman

Copy link
Copy Markdown
Contributor

Can it also have a regression test please?

@felipensp felipensp marked this pull request as ready for review October 11, 2025 11:26
@felipensp

Copy link
Copy Markdown
Member Author

Can it also have a regression test please?

Test added.

Comment thread vlib/v/tests/generic_calls/top_level_domain.v

@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.

Does the test need so many files?

@felipensp

felipensp commented Oct 12, 2025 via email

Copy link
Copy Markdown
Member Author

@spytheman

Copy link
Copy Markdown
Contributor

Yes. It just breaks when spliting in multiples files.

Thank you 🙇🏻 .

@spytheman spytheman merged commit fb338d2 into vlang:master Oct 12, 2025
82 checks passed
@felipensp felipensp deleted the markused_generic_array_param branch October 13, 2025 11:17
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.

Enum cases/values

3 participants