Page MenuHomePhabricator
Files F15382535

D7958.id18005.diff
No OneTemporary
Actions

D7958.id18005.diff
View Options

Index: src/lint/linter/ArcanistExternalLinter.php
===================================================================
--- src/lint/linter/ArcanistExternalLinter.php
+++ src/lint/linter/ArcanistExternalLinter.php
@@ -324,7 +324,7 @@
* @return string Command to execute the raw linter.
* @task exec
*/
- protected function getExecutableCommand() {
+ final protected function getExecutableCommand() {
$this->checkBinaryConfiguration();
$interpreter = null;
@@ -351,7 +351,7 @@
* @return string Composed flags.
* @task exec
*/
- protected function getCommandFlags() {
+ final protected function getCommandFlags() {
return csprintf(
'%C %C',
$this->getMandatoryFlags(),
@@ -359,7 +359,7 @@
}
- protected function buildFutures(array $paths) {
+ final protected function buildFutures(array $paths) {
$executable = $this->getExecutableCommand();
$bin = csprintf('%C %C', $executable, $this->getCommandFlags());
@@ -384,7 +384,7 @@
return $futures;
}
- protected function resolveFuture($path, Future $future) {
+ final protected function resolveFuture($path, Future $future) {
list($err, $stdout, $stderr) = $future->resolve();
if ($err && !$this->shouldExpectCommandErrors()) {
$future->resolvex();
Index: src/lint/linter/ArcanistLinter.php
===================================================================
--- src/lint/linter/ArcanistLinter.php
+++ src/lint/linter/ArcanistLinter.php
@@ -39,20 +39,20 @@
return $this;
}
- public function setConfig(array $config) {
+ final public function setConfig(array $config) {
$this->config = $config;
return $this;
}
- protected function getConfig($key, $default = null) {
+ final protected function getConfig($key, $default = null) {
return idx($this->config, $key, $default);
}
- public function getActivePath() {
+ final public function getActivePath() {
return $this->activePath;
}
- public function getOtherLocation($offset, $path = null) {
+ final public function getOtherLocation($offset, $path = null) {
if ($path === null) {
$path = $this->getActivePath();
}
@@ -68,21 +68,21 @@
);
}
- public function stopAllLinters() {
+ final public function stopAllLinters() {
$this->stopAllLinters = true;
return $this;
}
- public function didStopAllLinters() {
+ final public function didStopAllLinters() {
return $this->stopAllLinters;
}
- public function addPath($path) {
+ final public function addPath($path) {
$this->paths[$path] = $path;
return $this;
}
- public function setPaths(array $paths) {
+ final public function setPaths(array $paths) {
$this->paths = $paths;
return $this;
}
@@ -91,7 +91,7 @@
* Filter out paths which this linter doesn't act on (for example, because
* they are binaries and the linter doesn't apply to binaries).
*/
- private function filterPaths($paths) {
+ final private function filterPaths($paths) {
$engine = $this->getEngine();
$keep = array();
@@ -114,16 +114,16 @@
return $keep;
}
- public function getPaths() {
+ final public function getPaths() {
return $this->filterPaths(array_values($this->paths));
}
- public function addData($path, $data) {
+ final public function addData($path, $data) {
$this->data[$path] = $data;
return $this;
}
- protected function getData($path) {
+ final protected function getData($path) {
if (!array_key_exists($path, $this->data)) {
$this->data[$path] = $this->getEngine()->loadData($path);
}
@@ -135,7 +135,7 @@
return $this;
}
- protected function getEngine() {
+ final protected function getEngine() {
return $this->engine;
}
@@ -143,11 +143,11 @@
return 0;
}
- public function getLintMessageFullCode($short_code) {
+ final public function getLintMessageFullCode($short_code) {
return $this->getLinterName().$short_code;
}
- public function getLintMessageSeverity($code) {
+ final public function getLintMessageSeverity($code) {
$map = $this->customSeverityMap;
if (isset($map[$code])) {
return $map[$code];
@@ -171,7 +171,7 @@
return ArcanistLintSeverity::SEVERITY_ERROR;
}
- public function isMessageEnabled($code) {
+ final public function isMessageEnabled($code) {
return ($this->getLintMessageSeverity($code) !==
ArcanistLintSeverity::SEVERITY_DISABLED);
}
@@ -184,7 +184,7 @@
return "Unknown lint message!";
}
- protected function addLintMessage(ArcanistLintMessage $message) {
+ final protected function addLintMessage(ArcanistLintMessage $message) {
if (!$this->getEngine()->getCommitHookMode()) {
$root = $this->getEngine()->getWorkingCopy()->getProjectRoot();
$path = Filesystem::resolvePath($message->getPath(), $root);
@@ -194,11 +194,11 @@
return $message;
}
- public function getLintMessages() {
+ final public function getLintMessages() {
return $this->messages;
}
- protected function raiseLintAtLine(
+ final protected function raiseLintAtLine(
$line,
$char,
$code,
@@ -220,14 +220,14 @@
return $this->addLintMessage($message);
}
- protected function raiseLintAtPath(
+ final protected function raiseLintAtPath(
$code,
$desc) {
return $this->raiseLintAtLine(null, null, $code, $desc, null, null);
}
- protected function raiseLintAtOffset(
+ final protected function raiseLintAtOffset(
$offset,
$code,
$desc,
@@ -272,7 +272,7 @@
// This is a hook.
}
- protected function isCodeEnabled($code) {
+ final protected function isCodeEnabled($code) {
$severity = $this->getLintMessageSeverity($code);
return $this->getEngine()->isSeverityEnabled($severity);
}

File Metadata

Mime Type
text/plain
Expires
Sat, Mar 15, 12:54 PM (2 w, 6 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7693658
Default Alt Text
D7958.id18005.diff (5 KB)

Event Timeline

Phabricator · Built by Phacility