<p> Attention is currently required from: Joshua Colp. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/16232">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 apps/app_originate.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/+/16232/comment/b428b81e_a21c5bf1">Patch Set #2, Line 204:</a> <code style="font-family:monospace,monospace">      ast_format_cap_append(capabilities, ast_format_slin, 0);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">This is a behavior change to have this single codec. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The change was because a) people can now specify which codecs to use, including any slin codecs, and b) it seems that we have found that using Originate can consistently cause weird issues because slin192 will end up getting used, so this should "work better". In most cases, slin is desired over slin192 (at least from my assessment), so the default should be the most compatible way. This is also closer to the behavior of the Originate AMI action and call files, neither of which add all the slin's to their codec list.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/16232/comment/c0c87180_41a2ed94">Patch Set #2, Line 262:</a> <code style="font-family:monospace,monospace">                   ast_format_cap_update_by_allow_disallow(capabilities, opt_args[OPT_ARG_CODECS], 1);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Does this keep slin in the capabilties by appending what is allowed, or does it start fresh?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">It keeps slin and appends what is allowed, since a function call only adds to or removes. Another reason I kept the slin list to just slin, rather than all of them.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/16232">change 16232</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/+/16232"/><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: I96a1aeb83d54b635b7a51e1b4680f03791622883 </div>
<div style="display:none"> Gerrit-Change-Number: 16232 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 05 Aug 2021 12:20:58 +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"> Gerrit-MessageType: comment </div>