diff options
| author | 2016-08-15 12:32:51 -0700 | |
|---|---|---|
| committer | 2016-08-15 12:32:51 -0700 | |
| commit | 470f8fcb5c0c59b28b5f73ee011a797b05085da7 (patch) | |
| tree | 94b3c5f77f57476c0f314edb53a01e90220cbfb0 /tools/aapt2/process | |
| parent | af34bd5caeeaed956f3f6728a076799b11fe9406 (diff) | |
Fix clang-tidy warnings in aapt and aapt2.
* Add explicit keyword to conversion constructors.
* Add NOLINT(implicit) comments for implicit conversion constructors.
Bug: 28341362
* Use const reference type for read-only parameters.
Bug: 30407689
* Add NOLINT to suppress wrong clang-tidy warnings on macros.
Test: build with WITH_TIDY=1
Change-Id: Ibfafb23d9be463b4072745a51a6d9f1812dcac45
Diffstat (limited to 'tools/aapt2/process')
| -rw-r--r-- | tools/aapt2/process/SymbolTable.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/aapt2/process/SymbolTable.h b/tools/aapt2/process/SymbolTable.h index bd01b64281d5..bd31416a5cee 100644 --- a/tools/aapt2/process/SymbolTable.h +++ b/tools/aapt2/process/SymbolTable.h @@ -54,7 +54,7 @@ public: Symbol() : Symbol(Maybe<ResourceId>{}) { } - Symbol(const Maybe<ResourceId>& i) : Symbol(i, nullptr) { + explicit Symbol(const Maybe<ResourceId>& i) : Symbol(i, nullptr) { } Symbol(const Maybe<ResourceId>& i, const std::shared_ptr<Attribute>& attr) : |