diff options
author | Glenn Kasten <gkasten@google.com> | 2012-01-20 09:19:01 -0800 |
---|---|---|
committer | Glenn Kasten <gkasten@google.com> | 2012-02-24 17:00:30 -0800 |
commit | a3b09254d44cd8d66ec947abe547538c4cfeaa89 (patch) | |
tree | 9a21a4b2b34e264d71f4a44262e5f555ce9513d6 /services | |
parent | 37d825e72a6c606553a745da1212590a425996d3 (diff) | |
download | frameworks_av-a3b09254d44cd8d66ec947abe547538c4cfeaa89.zip frameworks_av-a3b09254d44cd8d66ec947abe547538c4cfeaa89.tar.gz frameworks_av-a3b09254d44cd8d66ec947abe547538c4cfeaa89.tar.bz2 |
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
Diffstat (limited to 'services')
-rw-r--r-- | services/audioflinger/AudioFlinger.cpp | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/services/audioflinger/AudioFlinger.cpp b/services/audioflinger/AudioFlinger.cpp index d522091..3f49941 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 <NotificationClient> 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(); |