commit 470f8fcb5c0c59b28b5f73ee011a797b05085da7
author Chih-Hung Hsieh <chh@google.com> Mon Aug 15 12:32:51 2016 -0700
committer Chih-Hung Hsieh <chh@google.com> Mon Aug 15 12:32:51 2016 -0700
tree 94b3c5f77f57476c0f314edb53a01e90220cbfb0
parent af34bd5caeeaed956f3f6728a076799b11fe9406

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

tools/aapt2/NameMangler.h

diff --git a/tools/aapt2/NameMangler.h b/tools/aapt2/NameMangler.h
index 505a982e..b6aaa4d 100644
--- a/tools/aapt2/NameMangler.h
+++ b/tools/aapt2/NameMangler.h
@@ -43,7 +43,7 @@
     NameManglerPolicy mPolicy;
 
 public:
-    NameMangler(NameManglerPolicy policy) : mPolicy(policy) {
+    explicit NameMangler(NameManglerPolicy policy) : mPolicy(policy) {
     }
 
     Maybe<ResourceName> mangleName(const ResourceName& name) {