<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">(1 comment)</p><p style="white-space: pre-wrap; word-wrap: break-word;">patch looks good, just not sure about the semantics. what was the<br>alt-codec intended for and why don't we use it after all?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I have no Idea what this alt_codec was about. When I get the things right then one could negotiate multiple codecs via SDP and then just use them. So basically we deal with multiple alternative codecs which we may switch afterwards and by the standard it may be definitely more than one alternative codec, thats why I do not get why there is only<br>a single alt codec and not multiple ones.</p><p><a href="https://gerrit.osmocom.org/9234">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/9234/1//COMMIT_MSG">Commit Message:</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.osmocom.org/#/c/9234/1//COMMIT_MSG@11">Patch Set #1, Line 11:</a> <code style="font-family:monospace,monospace">- remove unused alt_codec field</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">oh so there's a scheme behind repeating the same statements</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Unfortunately yes. Every commit has a title, an explaination and a bullet point summary. I am not sure if this is right or so. We do not have official rules for this, right?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/9234">change 9234</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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.osmocom.org/9234"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I5ff2899e3e61f33eb86f284b50ad8a94a949ed16 </div>
<div style="display:none"> Gerrit-Change-Number: 9234 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 24 May 2018 07:34:58 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>