summaryrefslogtreecommitdiff
path: root/tools
diff options
context:
space:
mode:
authorElliott Hughes <enh@google.com>2018-01-19 15:45:23 -0800
committerElliott Hughes <enh@google.com>2018-01-19 15:56:12 -0800
commit5cec377f49d6429b19895eb632225cd757ea611e (patch)
treebcd08fe1e6b656a5a4e235a225fbe88feb54a112 /tools
parent0d63a3c233040af004cc470d5f76547f3adc0148 (diff)
Address a bunch of clang-tidy complaints.
There were a bunch more unreasonable/incorrect ones, but these ones seemed legit. Nothing very interesting, though. Bug: N/A Test: ran tests, benchmarks Change-Id: If66971194d4a7b4bf6d0251bedb88e8cdc88a76f
Diffstat (limited to 'tools')
-rw-r--r--tools/versioner/src/DeclarationDatabase.cpp3
-rw-r--r--tools/versioner/src/versioner.cpp2
2 files changed, 0 insertions, 5 deletions
diff --git a/tools/versioner/src/DeclarationDatabase.cpp b/tools/versioner/src/DeclarationDatabase.cpp
index 19c2f0d0a..d9cff17d5 100644
--- a/tools/versioner/src/DeclarationDatabase.cpp
+++ b/tools/versioner/src/DeclarationDatabase.cpp
@@ -102,21 +102,18 @@ class Visitor : public RecursiveASTVisitor<Visitor> {
return true;
}
- DeclarationType declaration_type;
std::string declaration_name = getDeclName(named_decl);
bool is_extern = named_decl->getFormalLinkage() == ExternalLinkage;
bool is_definition = false;
bool no_guard = false;
if (auto function_decl = dyn_cast<FunctionDecl>(decl)) {
- declaration_type = DeclarationType::function;
is_definition = function_decl->isThisDeclarationADefinition();
} else if (auto var_decl = dyn_cast<VarDecl>(decl)) {
if (!var_decl->isFileVarDecl()) {
return true;
}
- declaration_type = DeclarationType::variable;
switch (var_decl->isThisDeclarationADefinition()) {
case VarDecl::DeclarationOnly:
is_definition = false;
diff --git a/tools/versioner/src/versioner.cpp b/tools/versioner/src/versioner.cpp
index efb39bd83..af0c06751 100644
--- a/tools/versioner/src/versioner.cpp
+++ b/tools/versioner/src/versioner.cpp
@@ -496,7 +496,6 @@ extern "C" const char* __asan_default_options() {
int main(int argc, char** argv) {
std::string cwd = getWorkingDir() + "/";
- bool default_args = true;
std::string platform_dir;
std::set<Arch> selected_architectures;
std::set<int> selected_levels;
@@ -507,7 +506,6 @@ int main(int argc, char** argv) {
int c;
while ((c = getopt(argc, argv, "a:r:p:so:fdj:vhFi")) != -1) {
- default_args = false;
switch (c) {
case 'a': {
char* end;