From 538529b9dcdb44c6aec631916f83611355b7ee8a Mon Sep 17 00:00:00 2001 From: Glenn Kasten Date: Fri, 20 Jan 2012 09:19:01 -0800 Subject: [PATCH] Simplify removeNotificationClient No need to check for presence of item before removing (but we do lose the log of the previous value). Change-Id: I2838430824de5f257f2ee15db0c22b1920c67d08 --- services/audioflinger/AudioFlinger.cpp | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/services/audioflinger/AudioFlinger.cpp b/services/audioflinger/AudioFlinger.cpp index d5220912b198a..3f4994183df60 100644 --- a/services/audioflinger/AudioFlinger.cpp +++ b/services/audioflinger/AudioFlinger.cpp @@ -1023,12 +1023,7 @@ void AudioFlinger::removeNotificationClient(pid_t pid) { Mutex::Autolock _l(mLock); - ssize_t index = mNotificationClients.indexOfKey(pid); - if (index >= 0) { - sp client = mNotificationClients.valueFor(pid); - ALOGV("removeNotificationClient() %p, pid %d", client.get(), pid); - mNotificationClients.removeItem(pid); - } + mNotificationClients.removeItem(pid); ALOGV("%d died, releasing its sessions", pid); size_t num = mAudioSessionRefs.size();