diff options
author | Nikita Popov <nikita.ppv@gmail.com> | 2021-01-25 16:17:14 +0100 |
---|---|---|
committer | Nikita Popov <nikita.ppv@gmail.com> | 2021-01-25 16:17:50 +0100 |
commit | 18507853cb727361ea09fca8a7e4458b9262b145 (patch) | |
tree | 2eb814e653df0b4933ec0338ff1a712cbe7d508c | |
parent | d319098b24342e9a4e2344dc7f588b74edcaa846 (diff) | |
download | php-git-18507853cb727361ea09fca8a7e4458b9262b145.tar.gz |
Improve switch continue warning
Don't suggest "continue N+1" if there is no wrapping loop. The
resulting code would be illegal.
-rw-r--r-- | Zend/tests/continue_targeting_switch_warning.phpt | 21 | ||||
-rw-r--r-- | Zend/zend_compile.c | 27 |
2 files changed, 38 insertions, 10 deletions
diff --git a/Zend/tests/continue_targeting_switch_warning.phpt b/Zend/tests/continue_targeting_switch_warning.phpt index 05a51e5a22..379108ad1f 100644 --- a/Zend/tests/continue_targeting_switch_warning.phpt +++ b/Zend/tests/continue_targeting_switch_warning.phpt @@ -22,6 +22,21 @@ function test() { } } + switch ($bar) { + case 0: + while ($xyz) { + continue 2; // INVALID + } + case 1: + while ($xyz) { + continue; + } + case 2: + while ($xyz) { + break 2; + } + } + while ($foo) { switch ($bar) { case 0: @@ -42,8 +57,10 @@ function test() { ?> --EXPECTF-- -Warning: "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in %s on line 6 +Warning: "continue" targeting switch is equivalent to "break" in %s on line 6 Warning: "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in %s on line 14 -Warning: "continue 2" targeting switch is equivalent to "break 2". Did you mean to use "continue 3"? in %s on line 26 +Warning: "continue 2" targeting switch is equivalent to "break 2" in %s on line 25 + +Warning: "continue 2" targeting switch is equivalent to "break 2". Did you mean to use "continue 3"? in %s on line 41 diff --git a/Zend/zend_compile.c b/Zend/zend_compile.c index e7f2b0b04a..57c33e3c96 100644 --- a/Zend/zend_compile.c +++ b/Zend/zend_compile.c @@ -4891,15 +4891,26 @@ void zend_compile_break_continue(zend_ast *ast) /* {{{ */ if (CG(context).brk_cont_array[cur].is_switch) { if (depth == 1) { - zend_error(E_WARNING, - "\"continue\" targeting switch is equivalent to \"break\". " \ - "Did you mean to use \"continue " ZEND_LONG_FMT "\"?", - depth + 1); + if (CG(context).brk_cont_array[cur].parent == -1) { + zend_error(E_WARNING, + "\"continue\" targeting switch is equivalent to \"break\""); + } else { + zend_error(E_WARNING, + "\"continue\" targeting switch is equivalent to \"break\". " \ + "Did you mean to use \"continue " ZEND_LONG_FMT "\"?", + depth + 1); + } } else { - zend_error(E_WARNING, - "\"continue " ZEND_LONG_FMT "\" targeting switch is equivalent to \"break " ZEND_LONG_FMT "\". " \ - "Did you mean to use \"continue " ZEND_LONG_FMT "\"?", - depth, depth, depth + 1); + if (CG(context).brk_cont_array[cur].parent == -1) { + zend_error(E_WARNING, + "\"continue " ZEND_LONG_FMT "\" targeting switch is equivalent to \"break " ZEND_LONG_FMT "\"", + depth, depth); + } else { + zend_error(E_WARNING, + "\"continue " ZEND_LONG_FMT "\" targeting switch is equivalent to \"break " ZEND_LONG_FMT "\". " \ + "Did you mean to use \"continue " ZEND_LONG_FMT "\"?", + depth, depth, depth + 1); + } } } } |