diff options
author | Ivan Komissarov <ABBAPOH@gmail.com> | 2019-11-24 16:14:53 +0900 |
---|---|---|
committer | Ivan Komissarov <ABBAPOH@gmail.com> | 2020-01-07 12:58:12 +0000 |
commit | 861c81ab8603a01c23e97543837670503538119e (patch) | |
tree | 2329f5c0e80db5db3bbe4b155853168643589078 /examples | |
parent | 375ba8b23f0bf2b8639e9d7196f4451149798464 (diff) |
Fix 'performance-unnecessary-value-param' warnings
Use more pass-by-const-ref since there is no point in passing objects by
value in these cases
Also, treat this warning as error
Change-Id: I5382dcd56639f535cf240205cc5c3ac667c4ece3
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Diffstat (limited to 'examples')
-rw-r--r-- | examples/grpc/client.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/examples/grpc/client.cpp b/examples/grpc/client.cpp index 55bcd866d..8fee915fb 100644 --- a/examples/grpc/client.cpp +++ b/examples/grpc/client.cpp @@ -51,7 +51,7 @@ class Client { public: - Client(std::shared_ptr<grpc::Channel> channel) + Client(const std::shared_ptr<grpc::Channel> &channel) : m_stub(PP::MyApi::NewStub(channel)) {} int ping(int count) { |