Page MenuHomePhabricator

Address continue statement scope
AbandonedPublic

Authored by epriestley on Oct 21 2018, 9:08 PM.
Tags
None
Referenced Files
F19498808: D19749.diff
Thu, Jan 8, 1:19 PM
F19189930: D19749.id47181.diff
Tue, Dec 16, 8:48 AM
F19098210: D19749.id.diff
Dec 4 2025, 4:07 PM
F19089064: D19749.diff
Dec 3 2025, 5:54 AM
F19014924: D19749.id47181.diff
Nov 22 2025, 9:58 PM
F18844325: D19749.id.diff
Oct 29 2025, 4:38 AM
F18738307: D19749.id47181.diff
Oct 1 2025, 3:03 PM
F18468996: D19749.id.diff
Sep 2 2025, 4:11 PM
Subscribers

Details

Reviewers
dereckson
Group Reviewers
Blessed Reviewers
Summary

PHP 7.3 throws an exception when a continue is used inside a switch.

As the code intent is to skip the rest of the current outer loop iteration
(the loop containing the switch), there is a need to specify the level.

Test Plan

arc lint

Diff Detail

Repository
rARC Arcanist
Branch
master
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 21026
Build 28562: arc lint + arc unit

Unit TestsFailed

TimeTest
392 msArcanistPhpcsLinterTestCase::Unknown Unit Message ("")
In 'basics.lint-test', expected lint to raise error on line 2 at char 1, but no error was raised. Actually raised: No messages.
5,163 msArcanistRuboCopLinterTestCase::Unknown Unit Message ("")
In 'convention.lint-test', expected lint to raise warning on line 1 at char 10, but no warning was raised. Actually raised: No messages.
367 msArcanistAbstractMethodBodyXHPASTLinterRuleTestCase::Unknown Unit Message ("")
7 assertions passed.
25 msArcanistAbstractPrivateMethodXHPASTLinterRuleTestCase::Unknown Unit Message ("")
3 assertions passed.
28 msArcanistAliasFunctionXHPASTLinterRuleTestCase::Unknown Unit Message ("")
4 assertions passed.
View Full Test Results (2 Failed · 126 Passed · 27 Skipped)