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/util/BigBuffer.h

diff --git a/tools/aapt2/util/BigBuffer.h b/tools/aapt2/util/BigBuffer.h
index cad2a2e..ba8532f 100644
--- a/tools/aapt2/util/BigBuffer.h
+++ b/tools/aapt2/util/BigBuffer.h
@@ -63,7 +63,7 @@
      * Create a BigBuffer with block allocation sizes
      * of blockSize.
      */
-    BigBuffer(size_t blockSize);
+    explicit BigBuffer(size_t blockSize);
 
     BigBuffer(const BigBuffer&) = delete; // No copying.