<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7670">View Change</a></p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm -1 on this patch series.  If it's OK to break these API's because they're unused I'd prefer we just remove them.  I don't want us to get in a habit of saying "well that API is unused so it's ok to break it".  Removal would be a one-time breakage, we wouldn't be making the exception next time master does something (this is still a new API, it's reasonable to expect changes).</p><p style="white-space: pre-wrap; word-wrap: break-word;">From IRC last night:<br><rmudgett> https://gerrit.asterisk.org/#/c/7670/ to https://gerrit.asterisk.org/#/c/7672/ are to bring v15 up to master where the new SDP code is concerned.<br><rmudgett> Though that code is effectively dead code since nothing uses it yet.<br><rmudgett> file: coreyfarrell gtjoseph ^^^<br><gtjoseph> ok<br><gtjoseph> i'm unsure of the implications.<br><gtjoseph> or whether there is an action item for me<br><rmudgett> Well they are reviews and this just keeps the master and v15 in sync to prevent unnecessary divergence.<br><gtjoseph> Oh, OK.<br><rmudgett> Otherwise, I think the new SDP code should be removed from v15.<br><coreyfarrell> rmudgett: I assume you are aware of the ABI/API changes?  if it's dead-code and we're OK with pulling it from an ABI/API standpoint I think that might be best.<br><rmudgett> Yep.  Nothing uses it yet so there isn't an API/ABI problem.<br><coreyfarrell> my vote is to remove it from 15.  if a few months from now we want it in 15 we can bring it back but more likely it'll be for Asterisk 16 and we'll avoid the work of keeping it in sync.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7670">change 7670</a>. To unsubscribe, 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/7670"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 15 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: If07fe6d79fbdce33968a9401d41d908385043a06 </div>
<div style="display:none"> Gerrit-Change-Number: 7670 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 20 Dec 2017 16:57:42 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>