AudioPolicy: Reuse SortedVector comparison from AudioProfile
Operation for comparing sorted vectors was defined twice in APM code:
once in AudioProfile.h as operator==, then in APM as 'vectorsEqual'
method. Remove the latter and use the operator instead.
Test: make
Change-Id: I0697843aabb13539b7b552773d96dbb7fb56daa4
diff --git a/services/audiopolicy/managerdefault/AudioPolicyManager.cpp b/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
index bc9514f..cd33fe1 100644
--- a/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
+++ b/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
@@ -3426,7 +3426,7 @@
return NO_ERROR;
}
-status_t AudioPolicyManager::stopAudioSource(audio_patch_handle_t handle __unused)
+status_t AudioPolicyManager::stopAudioSource(audio_patch_handle_t handle)
{
sp<AudioSourceDescriptor> sourceDesc = mAudioSources.valueFor(handle);
ALOGV("%s handle %d", __FUNCTION__, handle);
@@ -4637,20 +4637,6 @@
return outputs;
}
-bool AudioPolicyManager::vectorsEqual(SortedVector<audio_io_handle_t>& outputs1,
- SortedVector<audio_io_handle_t>& outputs2)
-{
- if (outputs1.size() != outputs2.size()) {
- return false;
- }
- for (size_t i = 0; i < outputs1.size(); i++) {
- if (outputs1[i] != outputs2[i]) {
- return false;
- }
- }
- return true;
-}
-
void AudioPolicyManager::checkForDeviceAndOutputChanges()
{
checkForDeviceAndOutputChanges([](){ return false; });
@@ -4691,7 +4677,7 @@
}
}
- if (!vectorsEqual(srcOutputs,dstOutputs)) {
+ if (srcOutputs != dstOutputs) {
// get maximum latency of all source outputs to determine the minimum mute time guaranteeing
// audio from invalidated tracks will be rendered when unmuting
uint32_t maxLatency = 0;