revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-10 08:52:39 +01:00
Code Issues Releases Wiki Activity

Log two times per linter instead of N

Browse source 
Summary: This number seems more interesting and it includes time for resolving futures which is the main part of some linters.

Test Plan:
  $ arc --trace lint

Reviewers: fdeliege, epriestley

Reviewed By: epriestley

CC: aran, epriestley, Korvin

Differential Revision: https://secure.phabricator.com/D5075
This commit is contained in:
vrana 2013-02-22 09:21:07 -08:00
parent e33bd82c42
commit b758e3737c
1 changed files with 23 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
src/lint/engine/ArcanistLintEngine.php
View file

@ -229,26 +229,27 @@ abstract class ArcanistLintEngine {
$paths = array_values($paths);
if ($paths) {
$linter->willLintPaths($paths);
$profiler = PhutilServiceProfiler::getInstance();
foreach ($paths as $path) {
$linter->willLintPath($path);
$call_id = $profiler->beginServiceCall(array(
'type' => 'lint',
'linter' => get_class($linter),
'path' => $path,
'paths' => $paths,
));
try {
$linter->willLintPaths($paths);
foreach ($paths as $path) {
$linter->willLintPath($path);
$linter->lintPath($path);
if ($linter->didStopAllLinters()) {
$this->stopped[$path] = $linter_name;
}
}
} catch (Exception $ex) {
$profiler->endServiceCall($call_id, array());
throw $ex;
}
$profiler->endServiceCall($call_id, array());
if ($linter->didStopAllLinters()) {
$this->stopped[$path] = $linter_name;
}
}
}
} catch (Exception $ex) {
@ -356,8 +357,15 @@ abstract class ArcanistLintEngine {
protected function didRunLinters(array $linters) {
assert_instances_of($linters, 'ArcanistLinter');
$profiler = PhutilServiceProfiler::getInstance();
foreach ($linters as $linter) {
$call_id = $profiler->beginServiceCall(array(
'type' => 'lint',
'linter' => get_class($linter),
));
$linter->didRunLinters();
$profiler->endServiceCall($call_id, array());
}
}