Page MenuHomePhabricator

Statically detect "continue" inside "switch"
ClosedPublic

Authored by epriestley on Dec 22 2018, 1:44 PM.
Tags
None
Referenced Files
F13097562: D19931.id47571.diff
Fri, Apr 26, 1:26 AM
Unknown Object (File)
Thu, Apr 25, 1:10 AM
Unknown Object (File)
Sat, Apr 20, 4:23 PM
Unknown Object (File)
Sat, Apr 20, 12:15 AM
Unknown Object (File)
Wed, Apr 17, 2:45 PM
Unknown Object (File)
Sun, Apr 7, 11:26 PM
Unknown Object (File)
Sat, Apr 6, 9:20 PM
Unknown Object (File)
Fri, Apr 5, 2:38 PM
Subscribers
None

Details

Summary

See 30 prior patches. This is a fatal in PHP7, let's just hunt these down.

Test Plan

Ran unit tests. See next diff for results.

Diff Detail

Repository
rARC Arcanist
Branch
lint1
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 21424
Build 29178: Run Core Tests
Build 29177: arc lint + arc unit

Unit TestsFailed

TimeTest
691 msArcanistJSHintLinterTestCase::Unknown Unit Message ("")
In 'jshint.lint-test', expected lint to raise error on line 9 at char 0, but no error was raised. Actually raised: warning at line 3, char 8: W033 JSHintW033 error at line 7, char 1: E019 JSHintE019
47 msArcanistAbstractMethodBodyXHPASTLinterRuleTestCase::Unknown Unit Message ("")
7 assertions passed.
29 msArcanistAbstractMethodBodyXHPASTLinterRuleTestCase::Unknown Unit Message ("")
7 assertions passed.
26 msArcanistAbstractPrivateMethodXHPASTLinterRuleTestCase::Unknown Unit Message ("")
3 assertions passed.
12 msArcanistAbstractPrivateMethodXHPASTLinterRuleTestCase::Unknown Unit Message ("")
3 assertions passed.
View Full Test Results (1 Failed · 294 Passed · 66 Skipped)