From aadaf9a795e5024cc99ded39c59665bb5c883a59 Mon Sep 17 00:00:00 2001 From: vrana Date: Fri, 22 Feb 2013 13:49:16 -0800 Subject: [PATCH] Speedup implicit fallthrough lint rule by 99.5 % Summary: At least on my sample file. Test Plan: Saw time 0.073 s instead of 12.606 s. Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D5086 --- src/lint/linter/ArcanistXHPASTLinter.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/lint/linter/ArcanistXHPASTLinter.php b/src/lint/linter/ArcanistXHPASTLinter.php index 0843ea81..886236be 100644 --- a/src/lint/linter/ArcanistXHPASTLinter.php +++ b/src/lint/linter/ArcanistXHPASTLinter.php @@ -534,9 +534,10 @@ final class ArcanistXHPASTLinter extends ArcanistBaseXHPASTLinter { $tokens = $block->getTokens(); $last = end($tokens); while ($last && $last = $last->getNextToken()) { - if (!$last->isSemantic()) { - $tokens[$last->getTokenID()] = $last; + if ($last->isSemantic()) { + break; } + $tokens[$last->getTokenID()] = $last; } $blocks[$key] = array(