<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">My previous change was about the core, main/channel.c. Furthermore, at that time, when I had fixed that, I did not understand all channels. This change here is about channels/chan_sip.c. It took some time and research to understand why this change is needed but was not required 13 years ago: There was a guard, introduced decades ago, which checked whether at least one format is offered/requested. However, that guard was wrongly limited to at least one *audio* format.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Different view: To make video+text work, this change here depends on the previous change. However, that previous change does not depend on this one. First, the core had to be fixed in general. Then, the channel chan_sip.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Finally, I did not withhold this change here. I finished it and its analysis just yesterday. At that time, coincidently, the previous change was ‘through’.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Does this answer your question? Or are you asking that such things should happen in one single change in future?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">No, that's fine.  There just was no explanation.  Can you add some comments to the commit message to that effect?<br></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/15379">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/15379">change 15379</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/+/15379"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: Ic16a3bf13cd1b5c4fc4041ed74961177d96b600f </div>
<div style="display:none"> Gerrit-Change-Number: 15379 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 28 Jan 2021 14:50:33 +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>