vieter-v
/
v
2
0
Fork 0
Code Issues Pull Requests Releases Activity

cgen,js: fix missing break in switch case, generated by match with range branches (#12260)

pull/12266/head
ChAoS_UnItY 2021-10-22 03:30:05 +08:00 committed by GitHub
parent da7dad07a3
commit 5b69593766
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
vlib/v/gen/c/cgen.v
View File

@ -4480,6 +4480,7 @@ fn (mut g Gen) match_expr_switch(node ast.MatchExpr, is_expr bool, cond_var stri
}
g.writeln(') {')
g.stmts_with_tmp_var(range_branch.stmts, tmp_var)
g.writeln('break;')
g.writeln('}')
}
g.indent--
@ -4543,6 +4544,7 @@ fn (mut g Gen) match_expr_switch(node ast.MatchExpr, is_expr bool, cond_var stri
}
g.writeln(') {')
g.stmts_with_tmp_var(range_branch.stmts, tmp_var)
g.writeln('break;')
g.writeln('}')
}
g.indent--

1
vlib/v/gen/c/testdata/enum_switch_case.out vendored 100644
View File

@ -0,0 +1 @@
4..5

10
vlib/v/gen/c/testdata/enum_switch_case.vv vendored
View File

@ -8,13 +8,17 @@ enum Enum {
}
fn main() {
e := Enum.e1
e := Enum.e5
match e {
.e1 {}
.e2 {}
.e3 {}
.e4 {}
4...5 {}
else {}
4...5 {
println('4..5')
}
else {
println('else')
}
}
}

2
vlib/v/gen/js/js.v
View File

@ -2362,6 +2362,7 @@ fn (mut g JsGen) match_expr_switch(node ast.MatchExpr, is_expr bool, cond_var Ma
}
g.writeln(') {')
g.stmts_with_tmp_var(range_branch.stmts, tmp_var)
g.writeln('break;')
g.writeln('}')
}
g.dec_indent()
@ -2423,6 +2424,7 @@ fn (mut g JsGen) match_expr_switch(node ast.MatchExpr, is_expr bool, cond_var Ma
}
g.writeln(') {')
g.stmts_with_tmp_var(range_branch.stmts, tmp_var)
g.writeln('break;')
g.writeln('}')
}
g.dec_indent()