diff options
author | Elliott Hughes <enh@google.com> | 2018-01-19 15:45:23 -0800 |
---|---|---|
committer | Elliott Hughes <enh@google.com> | 2018-01-19 15:56:12 -0800 |
commit | 5cec377f49d6429b19895eb632225cd757ea611e (patch) | |
tree | bcd08fe1e6b656a5a4e235a225fbe88feb54a112 /tools | |
parent | 0d63a3c233040af004cc470d5f76547f3adc0148 (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.cpp | 3 | ||||
-rw-r--r-- | tools/versioner/src/versioner.cpp | 2 |
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; |