commit | 5404ee174dd6b5e312d3a97bc63c2bc8d142012b | [log] [tgz] |
---|---|---|
author | Chih-Hung Hsieh <chh@google.com> | Tue Aug 09 14:25:53 2016 -0700 |
committer | Chih-hung Hsieh <chh@google.com> | Wed Aug 10 17:28:17 2016 +0000 |
tree | f91538f49c70a4730b3806617b1cdf11328e8b55 | |
parent | 67fbddfdc2cafd63c83d3bb55a2451d4c26f546d [diff] [blame] |
Fix clang-tidy warnings in camera. * Add explicit keyword to conversion constructors. Bug: 28341362 * Use const reference type for read-only parameters. Bug: 30407689 * Use const reference type to avoid unnecessary copy. Bug: 30413862 Test: build with WITH_TIDY=1 Change-Id: I71d3008da843ba5f1df1a73a320fb2af6ceffa16 Merged-In: I71d3008da843ba5f1df1a73a320fb2af6ceffa16
diff --git a/services/camera/libcameraservice/utils/ClientManager.h b/services/camera/libcameraservice/utils/ClientManager.h index 7ae58d5..830c50b 100644 --- a/services/camera/libcameraservice/utils/ClientManager.h +++ b/services/camera/libcameraservice/utils/ClientManager.h
@@ -216,7 +216,7 @@ static constexpr int32_t DEFAULT_MAX_COST = 100; ClientManager(); - ClientManager(int32_t totalCost); + explicit ClientManager(int32_t totalCost); /** * Add a given ClientDescriptor to the managed list. ClientDescriptors for clients that