Page MenuHomePhabricator

D9682.diff
No OneTemporary

D9682.diff

diff --git a/src/lint/linter/ArcanistCpplintLinter.php b/src/lint/linter/ArcanistCpplintLinter.php
--- a/src/lint/linter/ArcanistCpplintLinter.php
+++ b/src/lint/linter/ArcanistCpplintLinter.php
@@ -35,6 +35,10 @@
return $this->getDeprecatedConfiguration('lint.cpplint.options', array());
}
+ protected function getDefaultMessageSeverity($code) {
+ return ArcanistLintSeverity::SEVERITY_WARNING;
+ }
+
protected function parseLinterOutput($path, $err, $stdout, $stderr) {
$lines = explode("\n", $stderr);
@@ -49,13 +53,16 @@
foreach ($matches as $key => $match) {
$matches[$key] = trim($match);
}
+
+ $severity = $this->getLintMessageSeverity($matches[3]);
+
$message = new ArcanistLintMessage();
$message->setPath($path);
$message->setLine($matches[1]);
$message->setCode($matches[3]);
$message->setName($matches[3]);
$message->setDescription($matches[2]);
- $message->setSeverity(ArcanistLintSeverity::SEVERITY_WARNING);
+ $message->setSeverity($severity);
$messages[] = $message;
}

File Metadata

Mime Type
text/plain
Expires
Fri, Nov 8, 1:32 AM (4 d, 23 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6728321
Default Alt Text
D9682.diff (1 KB)

Event Timeline