commit 4f97a8ca543efa465a8e13ebcd689a08f95ea351
author Jiyong Park <jiyong@google.com> Mon Nov 16 18:11:14 2020 +0900
committer Jiyong Park <jiyong@google.com> Mon Nov 16 19:43:52 2020 +0900
tree 9840ccc8b805a992b1c3e9e18f1e75e5e9e48fd5
parent 16c6e7073231dc9e9a775acb5aa5c456ca38851f

binder_ndk: fix performance-unnecessary-value-param

Bug: 162909698
Test: m libbinder

Change-Id: I89779ab38a28972c1ad15733aff34a51e7e7fa8a

libs/binder/ndk/ibinder.cpp

diff --git a/libs/binder/ndk/ibinder.cpp b/libs/binder/ndk/ibinder.cpp
index b927f6f..5e2e1bd 100644
--- a/libs/binder/ndk/ibinder.cpp
+++ b/libs/binder/ndk/ibinder.cpp
@@ -367,7 +367,7 @@
                            mDeathRecipients.end());
 }
 
-binder_status_t AIBinder_DeathRecipient::linkToDeath(sp<IBinder> binder, void* cookie) {
+binder_status_t AIBinder_DeathRecipient::linkToDeath(const sp<IBinder>& binder, void* cookie) {
     CHECK(binder != nullptr);
 
     std::lock_guard<std::mutex> l(mDeathRecipientsMutex);
@@ -386,7 +386,7 @@
     return STATUS_OK;
 }
 
-binder_status_t AIBinder_DeathRecipient::unlinkToDeath(sp<IBinder> binder, void* cookie) {
+binder_status_t AIBinder_DeathRecipient::unlinkToDeath(const sp<IBinder>& binder, void* cookie) {
     CHECK(binder != nullptr);
 
     std::lock_guard<std::mutex> l(mDeathRecipientsMutex);