<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">This patch is huge, I'd welcome some splitting into several patches if possible.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I fully agree, and I tried that. The problem: it is hard to make parts of it work without the rest. After hitting complex problems trying to split things out, I flinched and left it in one.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I can spend (probably a lot of) time on separate patches if that is required, or we can agree on another "medium-large refactoring" bomb...<br>If the opinion is that it should be split, I can try harder.</p><p style="white-space: pre-wrap; word-wrap: break-word;">For example, I wanted to split out the part that moves the CN side CRCX to an earlier time, but that also entails changing the the callback functions' names and the sequence of what calls which. Then I would need to sort of undo the part that figures out the codecs from SDP, which IIRC changes the code flow and needs more thinking to get a fully working osmo-msc. We would get an intermediate stage osmo-msc that no-one will be using in practice and which probably has odd problems that are fixed by the rest of this patch; I'd need to fix those in creative ways...</p><p style="white-space: pre-wrap; word-wrap: break-word;">I could split out the SDP parsing stuff, but then that would have no callers.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I could split out the Codec List (BSS Supported) handling, but I store the result in the new cc_sdp struct. I'd have to invent some other storage for that and undo it in the second part of the patch. Or I could split it out without using it anywhere. Ah, I did that already in I66c735c79e982388f06b5de783aa584c9d13569e ...</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm open for opinions here.</p><p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/15953">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-msc/+/15953">change 15953</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-msc/+/15953"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I8c3b2de53ffae4ec3a66b9dabf308c290a2c999f </div>
<div style="display:none"> Gerrit-Change-Number: 15953 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 04 Nov 2019 14:24:54 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>