<p>Jenkins2 <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/7821">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Richard Mudgett: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bridge_softmix: Removed unused parameter from check_binaural_position_change(.).<br><br>Found as a result of the function being passed an uninitalized variable by<br>clang.<br><br>ASTERISK-27550<br><br>Change-Id: I8af3bd84656b685a956d498459f8db3613f68954<br>---<br>M bridges/bridge_softmix.c<br>M bridges/bridge_softmix/bridge_softmix_binaural.c<br>M bridges/bridge_softmix/include/bridge_softmix_internal.h<br>3 files changed, 4 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/bridges/bridge_softmix.c b/bridges/bridge_softmix.c<br>index 83b1a83..3343569 100644<br>--- a/bridges/bridge_softmix.c<br>+++ b/bridges/bridge_softmix.c<br>@@ -1404,7 +1404,7 @@<br>             }<br> <br> #ifdef BINAURAL_RENDERING<br>-             check_binaural_position_change(bridge, softmix_data, bridge_channel);<br>+                check_binaural_position_change(bridge, softmix_data);<br> #endif<br> <br>             /* Go through pulling audio from each factory that has it available */<br>diff --git a/bridges/bridge_softmix/bridge_softmix_binaural.c b/bridges/bridge_softmix/bridge_softmix_binaural.c<br>index 128e464..2a8efd7 100644<br>--- a/bridges/bridge_softmix/bridge_softmix_binaural.c<br>+++ b/bridges/bridge_softmix/bridge_softmix_binaural.c<br>@@ -465,9 +465,10 @@<br> }<br> <br> void check_binaural_position_change(struct ast_bridge *bridge,<br>-                struct softmix_bridge_data *softmix_data, struct ast_bridge_channel *bridge_channel)<br>+         struct softmix_bridge_data *softmix_data)<br> {<br>         unsigned int pos_change;<br>+     struct ast_bridge_channel *bridge_channel;<br> <br>         /*<br>     * We only check binaural things if binaural is activated by the config<br>diff --git a/bridges/bridge_softmix/include/bridge_softmix_internal.h b/bridges/bridge_softmix/include/bridge_softmix_internal.h<br>index 01e65aa..f842acb 100644<br>--- a/bridges/bridge_softmix/include/bridge_softmix_internal.h<br>+++ b/bridges/bridge_softmix/include/bridge_softmix_internal.h<br>@@ -377,10 +377,9 @@<br>  *<br>  * \param bridge The conference bridge.<br>  * \param softmix_data The data used by the softmix bridge.<br>- * \param bridge_channel The bridge channel.<br>  */<br> void check_binaural_position_change(struct ast_bridge *bridge,<br>-            struct softmix_bridge_data *softmix_data, struct ast_bridge_channel *bridge_channel);<br>+                struct softmix_bridge_data *softmix_data);<br> <br> /*!<br>  * \brief Processes audio data with the binaural synthesis and adds the result to the mixing array.<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7821">change 7821</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/7821"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 15 </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I8af3bd84656b685a956d498459f8db3613f68954 </div>
<div style="display:none"> Gerrit-Change-Number: 7821 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>