Skip to content

v: add in expr with numbers (var in 1..4) - #22754

Merged
spytheman merged 13 commits into
vlang:masterfrom
felipensp:add_in_expr_with_int
Nov 5, 2024
Merged

spytheman merged 13 commits into
vlang:masterfrom
felipensp:add_in_expr_with_int

Conversation

@felipensp

@felipensp felipensp commented Nov 4, 2024 •

Copy link
Copy Markdown
Member

@felipensp
felipensp marked this pull request as ready for review November 4, 2024 15:33
Comment thread vlib/v/tests/range_expr_with_int_test.v
Comment thread vlib/v/gen/c/infix.v Outdated
Comment thread vlib/v/gen/c/infix.v Outdated
Comment thread vlib/v/tests/range_expr_with_int_test.v Outdated
Comment thread vlib/v/tests/range_expr_with_int_test.v Outdated
Comment thread vlib/v/tests/range_expr_with_int_test.v Outdated
Comment thread vlib/v/tests/range_expr_with_int_test.v Outdated
Comment thread vlib/v/tests/range_expr_with_int_test.v Outdated
felipensp and others added 6 commits November 5, 2024 06:41
assert y in 4..20
assert y !in min..max

assert (t() !in 1..3) == false

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.

Suggested change
assert (t() !in 1..3) == false
assert t() in 1..3

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

@spytheman
spytheman merged commit 07eed88 into vlang:master Nov 5, 2024
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.

Implement if num in start..end

2 participants