<div dir="ltr"><div dir="ltr">On Wed, Sep 7, 2022 at 8:56 AM Fridrich Maximilian <<a href="mailto:M.Fridrich@commend.com">M.Fridrich@commend.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I have a potential fix that consists of two relatively separate changes, one of<br>
them modifies channel.c:ast_channel_request_stream_topology_change() and the<br>
other modififies res_pjsip_session.c:resolve_refresh_media_states() (both very<br>
small in size).<br>
<br>
Since one of them touches the core and both probably require some discussion,<br>
would you prefer it if I openend separate reviews on Gerrit for them? Only in<br>
concert they fix ASTERISK-30184 but I am unsure of some unintended consequences<br>
since they're relatively major changes of the logic.<br></blockquote><div><br></div><div>If they're fixing the same thing, then the same review is best in case it is ultimately merged and needs to be reverted. </div><div><br></div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div style="font-family:tahoma,sans-serif"><font color="#073763">Joshua C. Colp</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Asterisk Project Lead</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Sangoma Technologies</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Check us out at <a href="http://www.sangoma.com" target="_blank">www.sangoma.com</a> and <a href="http://www.asterisk.org" target="_blank">www.asterisk.org</a></font><br></div></div></div></div></div></div></div></div></div></div></div>