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

Fix visibility of future linter methods

Browse source
This commit is contained in:
vrana 2013-02-19 13:09:34 -08:00
parent 81171ca39d
commit 688e159319
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/lint/linter/ArcanistFutureLinter.php
View file

@ -4,8 +4,8 @@ abstract class ArcanistFutureLinter extends ArcanistLinter {
private $futures; private $futures;
abstract function buildFutures(array $paths); abstract protected function buildFutures(array $paths);
abstract function resolveFuture($path, Future $future); abstract protected function resolveFuture($path, Future $future);
public function willLintPaths(array $paths) { public function willLintPaths(array $paths) {
$this->futures = $this->buildFutures($paths); $this->futures = $this->buildFutures($paths);

4
src/lint/linter/ArcanistPEP8Linter.php
View file

@ -77,7 +77,7 @@ final class ArcanistPEP8Linter extends ArcanistFutureLinter {
return $bin; return $bin;
} }
public function buildFutures(array $paths) { protected function buildFutures(array $paths) {
$severity = ArcanistLintSeverity::SEVERITY_WARNING; $severity = ArcanistLintSeverity::SEVERITY_WARNING;
if (!$this->getEngine()->isSeverityEnabled($severity)) { if (!$this->getEngine()->isSeverityEnabled($severity)) {
return; return;
@ -101,7 +101,7 @@ final class ArcanistPEP8Linter extends ArcanistFutureLinter {
return $futures; return $futures;
} }
public function resolveFuture($path, Future $future) { protected function resolveFuture($path, Future $future) {
list($rc, $stdout) = $future->resolve(); list($rc, $stdout) = $future->resolve();
$lines = explode("\n", $stdout); $lines = explode("\n", $stdout);
$messages = array(); $messages = array();