mirror of
https://github.com/facebook/rocksdb.git
synced 2024-12-02 01:16:16 +00:00
d0458469c8
Summary: After we reached a consensus on code format, which follows exactly Google's coding style, a natural follow-up is to have a style checker that can handle stuffs beyond format. Google already has a powerful style checker "cpplint.py" and, luckily, phabricator already provides the built-in linter for it! Next time with "arc lint" most style inconsistency will be detected (but will not be fixed). Also I copied cpplint.py to linters directory, which is mostly because we may need the flexibility to make some modifications on it for our own need. Test Plan: ran arc lint table/block_based_table_builder.cc to see the amazing results. Reviewers: haobo, sdong, igor, dhruba Reviewed By: haobo CC: leveldb Differential Revision: https://reviews.facebook.net/D15369
148 lines
4.4 KiB
PHP
148 lines
4.4 KiB
PHP
<?php
|
|
// Copyright 2004-present Facebook. All rights reserved.
|
|
|
|
class FacebookFbcodeLintEngine extends ArcanistLintEngine {
|
|
|
|
public function buildLinters() {
|
|
$linters = array();
|
|
$paths = $this->getPaths();
|
|
|
|
// Remove all deleted files, which are not checked by the
|
|
// following linters.
|
|
foreach ($paths as $key => $path) {
|
|
if (!Filesystem::pathExists($this->getFilePathOnDisk($path))) {
|
|
unset($paths[$key]);
|
|
}
|
|
}
|
|
|
|
$generated_linter = new ArcanistGeneratedLinter();
|
|
$linters[] = $generated_linter;
|
|
|
|
$nolint_linter = new ArcanistNoLintLinter();
|
|
$linters[] = $nolint_linter;
|
|
|
|
$text_linter = new ArcanistTextLinter();
|
|
$text_linter->setCustomSeverityMap(array(
|
|
ArcanistTextLinter::LINT_LINE_WRAP
|
|
=> ArcanistLintSeverity::SEVERITY_ADVICE,
|
|
));
|
|
$linters[] = $text_linter;
|
|
|
|
$java_text_linter = new ArcanistTextLinter();
|
|
$java_text_linter->setMaxLineLength(100);
|
|
$java_text_linter->setCustomSeverityMap(array(
|
|
ArcanistTextLinter::LINT_LINE_WRAP
|
|
=> ArcanistLintSeverity::SEVERITY_ADVICE,
|
|
));
|
|
$linters[] = $java_text_linter;
|
|
|
|
$pep8_options = $this->getPEP8WithTextOptions().',E302';
|
|
|
|
$python_linter = new ArcanistPEP8Linter();
|
|
$python_linter->setConfig(array('options' => $pep8_options));
|
|
$linters[] = $python_linter;
|
|
|
|
$python_2space_linter = new ArcanistPEP8Linter();
|
|
$python_2space_linter->setConfig(array('options' => $pep8_options.',E111'));
|
|
$linters[] = $python_2space_linter;
|
|
|
|
// Currently we can't run cpplint in commit hook mode, because it
|
|
// depends on having access to the working directory.
|
|
if (!$this->getCommitHookMode()) {
|
|
$cpp_linters = array();
|
|
$google_linter = new ArcanistCpplintLinter();
|
|
$google_linter->setConfig(array(
|
|
'lint.cpplint.prefix' => '',
|
|
'lint.cpplint.bin' => 'cpplint',
|
|
));
|
|
$cpp_linters[] = $linters[] = $google_linter;
|
|
$cpp_linters[] = $linters[] = new FbcodeCppLinter();
|
|
$cpp_linters[] = $linters[] = new PfffCppLinter();
|
|
}
|
|
|
|
$spelling_linter = new ArcanistSpellingLinter();
|
|
$linters[] = $spelling_linter;
|
|
|
|
foreach ($paths as $path) {
|
|
$is_text = false;
|
|
|
|
$text_extensions = (
|
|
'/\.('.
|
|
'cpp|cxx|c|cc|h|hpp|hxx|tcc|'.
|
|
'py|rb|hs|pl|pm|tw|'.
|
|
'php|phpt|css|js|'.
|
|
'java|'.
|
|
'thrift|'.
|
|
'lua|'.
|
|
'siv|'.
|
|
'txt'.
|
|
')$/'
|
|
);
|
|
if (preg_match($text_extensions, $path)) {
|
|
$is_text = true;
|
|
}
|
|
if ($is_text) {
|
|
$nolint_linter->addPath($path);
|
|
|
|
$generated_linter->addPath($path);
|
|
$generated_linter->addData($path, $this->loadData($path));
|
|
|
|
if (preg_match('/\.java$/', $path)) {
|
|
$java_text_linter->addPath($path);
|
|
$java_text_linter->addData($path, $this->loadData($path));
|
|
} else {
|
|
$text_linter->addPath($path);
|
|
$text_linter->addData($path, $this->loadData($path));
|
|
}
|
|
|
|
$spelling_linter->addPath($path);
|
|
$spelling_linter->addData($path, $this->loadData($path));
|
|
}
|
|
if (preg_match('/\.(cpp|c|cc|cxx|h|hh|hpp|hxx|tcc)$/', $path)) {
|
|
foreach ($cpp_linters as &$linter) {
|
|
$linter->addPath($path);
|
|
$linter->addData($path, $this->loadData($path));
|
|
}
|
|
}
|
|
|
|
// Match *.py and contbuild config files
|
|
if (preg_match('/(\.(py|tw|smcprops)|^contbuild\/configs\/[^\/]*)$/',
|
|
$path)) {
|
|
$space_count = 4;
|
|
$real_path = $this->getFilePathOnDisk($path);
|
|
$dir = dirname($real_path);
|
|
do {
|
|
if (file_exists($dir.'/.python2space')) {
|
|
$space_count = 2;
|
|
break;
|
|
}
|
|
$dir = dirname($dir);
|
|
} while ($dir != '/' && $dir != '.');
|
|
|
|
if ($space_count == 4) {
|
|
$cur_path_linter = $python_linter;
|
|
} else {
|
|
$cur_path_linter = $python_2space_linter;
|
|
}
|
|
$cur_path_linter->addPath($path);
|
|
$cur_path_linter->addData($path, $this->loadData($path));
|
|
|
|
if (preg_match('/\.tw$/', $path)) {
|
|
$cur_path_linter->setCustomSeverityMap(array(
|
|
'E251' => ArcanistLintSeverity::SEVERITY_DISABLED,
|
|
));
|
|
}
|
|
}
|
|
}
|
|
|
|
$name_linter = new ArcanistFilenameLinter();
|
|
$linters[] = $name_linter;
|
|
foreach ($paths as $path) {
|
|
$name_linter->addPath($path);
|
|
}
|
|
|
|
return $linters;
|
|
}
|
|
|
|
}
|