commit 8d78f0b7b57a9eb88620c0961769609bb0430706
author Chih-Hung Hsieh <chh@google.com> Wed Jul 27 10:53:53 2016 -0700
committer Chih-Hung Hsieh <chh@google.com> Wed Jul 27 15:45:46 2016 -0700
tree 667054254f505d303330379bfa5665fb049de0e4
parent 69ed5d9a8306ee0427f378abe85cabfc5c3c1f17

Fix clang-tidy performance warnings in libhardware.

* Use const reference parameter type to avoid unnecessary copy.

Bug: 30407689
Change-Id: Ibf19e5529e548bba21fcc8403eb356cb49548172
Test: build with WITH_TIDY=1

tests/camera2/CameraMultiStreamTests.cpp

diff --git a/tests/camera2/CameraMultiStreamTests.cpp b/tests/camera2/CameraMultiStreamTests.cpp
index eb64db1..f4aeab7 100644
--- a/tests/camera2/CameraMultiStreamTests.cpp
+++ b/tests/camera2/CameraMultiStreamTests.cpp
@@ -334,7 +334,7 @@
             int height,
             const sp<CameraDeviceBase>& device,
             CameraStreamParams param = DEFAULT_STREAM_PARAMETERS,
-            sp<Surface> surface = NULL,
+            const sp<Surface>& surface = NULL,
             bool useCpuConsumer = true) {
         param.mFormat = MapAutoFormat(param.mFormat);
         return new CameraStream(width, height, device,