<p>neels <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/15132">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mgcp_codec: split codec_free() off of codec_init()<br><br>Both are used only in the same .c file, so make them static.<br><br>Move codec_set() guts into codec_add(): codec_set is only called by codec_add.<br>If codec_set were left separate, it'd look like the codec_init() is a bug and<br>lacks a codec_free() first. When looking at the entire context in codec_add(),<br>it becomes obvious that codec_init() should be called, not codec_free(),<br>because it is populating a previously unused entry.<br><br>Preparation to fix a memleak in a conn's codec list.<br><br>Change-Id: I120cab0a352a1e7b31c8f9c720c47b2c291311d7<br>---<br>M src/libosmo-mgcp/mgcp_codec.c<br>1 file changed, 35 insertions(+), 39 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/32/15132/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/15132">change 15132</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/c/osmo-mgw/+/15132"/><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-Change-Id: I120cab0a352a1e7b31c8f9c720c47b2c291311d7 </div>
<div style="display:none"> Gerrit-Change-Number: 15132 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>