<p>Sean Bright has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14899">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">format_cap: Include sample rates in format cap lists<br><br>ASTERISK-28416 #close<br><br>Change-Id: I9bf336acb9e703d7e9851142c162eaea93fc09e6<br>---<br>M main/format_cap.c<br>1 file changed, 14 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/99/14899/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/main/format_cap.c b/main/format_cap.c</span><br><span>index 86e8e1f..8a65c9e 100644</span><br><span>--- a/main/format_cap.c</span><br><span>+++ b/main/format_cap.c</span><br><span>@@ -748,12 +748,24 @@</span><br><span>  for (i = 0; i < AST_VECTOR_SIZE(&cap->preference_order); ++i) {</span><br><span>            int res;</span><br><span>             struct format_cap_framed *framed = AST_VECTOR_GET(&cap->preference_order, i);</span><br><span style="color: hsl(120, 100%, 40%);">+          const char *name = ast_format_get_name(framed->format);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-          res = ast_str_append(buf, 0, "%s%s", ast_format_get_name(framed->format),</span><br><span style="color: hsl(0, 100%, 40%);">-                  i < AST_VECTOR_SIZE(&cap->preference_order) - 1 ? "|" : "");</span><br><span style="color: hsl(120, 100%, 40%);">+            res = ast_str_append(buf, 0, "%s%s", i ? "|" : "", name);</span><br><span>              if (res < 0) {</span><br><span>                    break;</span><br><span>               }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+           /* Duplicated logic from main/translate.c - maybe we need a flag</span><br><span style="color: hsl(120, 100%, 40%);">+               * for this on the codec to make this less fragile? */</span><br><span style="color: hsl(120, 100%, 40%);">+                if (!strcmp(name, "slin") ||</span><br><span style="color: hsl(120, 100%, 40%);">+                        !strcmp(name, "speex") ||</span><br><span style="color: hsl(120, 100%, 40%);">+                   !strcmp(name, "silk")) {</span><br><span style="color: hsl(120, 100%, 40%);">+                    res = ast_str_append(buf, 0, "%u",</span><br><span style="color: hsl(120, 100%, 40%);">+                          ast_format_get_sample_rate(framed->format) / 1000);</span><br><span style="color: hsl(120, 100%, 40%);">+                        if (res < 0) {</span><br><span style="color: hsl(120, 100%, 40%);">+                             break;</span><br><span style="color: hsl(120, 100%, 40%);">+                        }</span><br><span style="color: hsl(120, 100%, 40%);">+             }</span><br><span>    }</span><br><span>    ast_str_append(buf, 0, ")");</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14899">change 14899</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/+/14899"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-Change-Id: I9bf336acb9e703d7e9851142c162eaea93fc09e6 </div>
<div style="display:none"> Gerrit-Change-Number: 14899 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>