<p> Attention is currently required from: Joshua Colp, Kevin Harwell. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/15953">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File include/asterisk/translate.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/15953/comment/1a773522_2a34ce3b">Patch Set #1, Line 285:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">/*! \brief Determine which protocol is higher quality<br> *<br> * Given two codec identifiers, is the first codec strictly superior to the second?<br> *<br> * \param codec identifier A<br> * \param codec identifier B<br> * \return Returns 1 if codec A is positively superior to B, 0 otherwise.<br> */<br>int ast_translator_better_quality(int c1, int c2);<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Since this function deals with choosing the best codec supplied by given formats I'd move this funct […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ack</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File main/translate.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/15953/comment/17e34330_b9203f30">Patch Set #2, Line 1385:</a> <code style="font-family:monospace,monospace">int ast_translator_better_quality(struct ast_format *src, struct ast_format *best) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I don't think this should be accepted as implemented. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done:</p><p style="white-space: pre-wrap; word-wrap: break-word;">[Oct  1 19:02:53]     -- Executing [s@c2:2] Playback("Local/s@c1-0000000b;1", "digits/4") in new stack<br>[Oct  1 19:02:53] DEBUG[898359][C-00000006]: translate.c:1491 ast_translator_best_choice: Tiebreaker: preferring format ulaw (100) to gsm (60)<br>[Oct  1 19:02:53] DEBUG[898359][C-00000006]: channel.c:5824 set_format: Channel Local/s@c1-0000000b;1 setting write format path: ulaw -> slin192<br>[Oct  1 19:02:53] DEBUG[898359][C-00000006]: channel.c:3207 ast_settimeout_full: Scheduling timer at (50 requested / 50 actual) timer ticks per second<br>[Oct  1 19:02:53]     -- <Local/s@c1-0000000b;1> Playing 'digits/4.ulaw' (language 'en')</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/15953">change 15953</a>. To unsubscribe, or for help writing mail filters, 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/c/asterisk/+/15953"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4b7297e1baca7aac14fe4a3c7538e18e2dbe9fd6 </div>
<div style="display:none"> Gerrit-Change-Number: 15953 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <default.enum@gmail.com> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: Kevin Harwell <default.enum@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 01 Oct 2022 23:14:00 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Comment-In-Reply-To: Kevin Harwell <default.enum@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>