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/common/CameraModule.h b/services/camera/libcameraservice/common/CameraModule.h index 36822c7..dbfaede 100644 --- a/services/camera/libcameraservice/common/CameraModule.h +++ b/services/camera/libcameraservice/common/CameraModule.h
@@ -32,7 +32,7 @@ */ class CameraModule { public: - CameraModule(camera_module_t *module); + explicit CameraModule(camera_module_t *module); virtual ~CameraModule(); // Must be called after construction