diff options
| author | Chih-Hung Hsieh <chh@google.com> | 2016-08-15 12:32:51 -0700 |
|---|---|---|
| committer | Chih-Hung Hsieh <chh@google.com> | 2016-08-15 12:32:51 -0700 |
| commit | 470f8fcb5c0c59b28b5f73ee011a797b05085da7 (patch) | |
| tree | 94b3c5f77f57476c0f314edb53a01e90220cbfb0 /tools/aapt2/Source.h | |
| 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/Source.h')
| -rw-r--r-- | tools/aapt2/Source.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/aapt2/Source.h b/tools/aapt2/Source.h index 319528e0ea1b..8a1021d49c39 100644 --- a/tools/aapt2/Source.h +++ b/tools/aapt2/Source.h @@ -35,7 +35,7 @@ struct Source { Source() = default; - inline Source(const StringPiece& path) : path(path.toString()) { + inline Source(const StringPiece& path) : path(path.toString()) { // NOLINT(implicit) } inline Source(const StringPiece& path, size_t line) : path(path.toString()), line(line) { |