commit 09f9c0237236f21cf331bcd811da89c26fc9c627
author Chih-Hung Hsieh <chh@google.com> Fri Jul 27 10:22:35 2018 -0700
committer Chih-Hung Hsieh <chh@google.com> Fri Jul 27 10:22:35 2018 -0700
tree a0a580f9c4f2270437739fc9ab8d1c46d073e4e1
parent 93adab132cfdd6e71cee2af14eea711b49a4c76d

Fix undefined error in ALOGV_IF.

Error found with clang static analyzer after
making ALOGV_IF to use its parameters.

Bug: 111850071
Test: build with WITH_TIDY
Change-Id: I35bdfd6583a106d82f30063cbd629bcff1d58289

media/libaudioprocessing/AudioMixer.cpp

diff --git a/media/libaudioprocessing/AudioMixer.cpp b/media/libaudioprocessing/AudioMixer.cpp
index f6f817a..f3ea826 100644
--- a/media/libaudioprocessing/AudioMixer.cpp
+++ b/media/libaudioprocessing/AudioMixer.cpp
@@ -875,7 +875,7 @@
                 t->hook = Track::getTrackHook(TRACKTYPE_RESAMPLE, t->mMixerChannelCount,
                         t->mMixerInFormat, t->mMixerFormat);
                 ALOGV_IF((n & NEEDS_CHANNEL_COUNT__MASK) > NEEDS_CHANNEL_2,
-                        "Track %d needs downmix + resample", i);
+                        "Track %d needs downmix + resample", name);
             } else {
                 if ((n & NEEDS_CHANNEL_COUNT__MASK) == NEEDS_CHANNEL_1){
                     t->hook = Track::getTrackHook(
@@ -890,7 +890,7 @@
                     t->hook = Track::getTrackHook(TRACKTYPE_NORESAMPLE, t->mMixerChannelCount,
                             t->mMixerInFormat, t->mMixerFormat);
                     ALOGV_IF((n & NEEDS_CHANNEL_COUNT__MASK) > NEEDS_CHANNEL_2,
-                            "Track %d needs downmix", i);
+                            "Track %d needs downmix", name);
                 }
             }
         }