From eb732555a71a3a1b15596dd65b6dad3c2e2af90b Mon Sep 17 00:00:00 2001
From: epriestley <git@epriestley.com>
Date: Wed, 12 Dec 2018 05:15:17 -0800
Subject: [PATCH] Fix a "continue;" inside a switch in
 ArcanistPHPCompatibilityXHPASTLinterRule

Summary: See <https://discourse.phabricator-community.org/t/arc-diff-fails-on-incompatible-linter-rule-with-php7-3/2198>.

Test Plan: Looked at the code carefully.

Reviewers: amckinley

Reviewed By: amckinley

Differential Revision: https://secure.phabricator.com/D19868
---
 .../xhpast/rules/ArcanistPHPCompatibilityXHPASTLinterRule.php   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lint/linter/xhpast/rules/ArcanistPHPCompatibilityXHPASTLinterRule.php b/src/lint/linter/xhpast/rules/ArcanistPHPCompatibilityXHPASTLinterRule.php
index d73d1fae..b2fa6770 100644
--- a/src/lint/linter/xhpast/rules/ArcanistPHPCompatibilityXHPASTLinterRule.php
+++ b/src/lint/linter/xhpast/rules/ArcanistPHPCompatibilityXHPASTLinterRule.php
@@ -70,7 +70,7 @@ final class ArcanistPHPCompatibilityXHPASTLinterRule
           $symbol = $params->getChildByIndex(0);
 
           if (!$symbol->isStaticScalar()) {
-            continue;
+            break;
           }
 
           $symbol_name = $symbol->evalStatic();