On Wed, Sep 7, 2022 at 8:56 AM Fridrich Maximilian <m.fridr...@commend.com>
wrote:

> I have a potential fix that consists of two relatively separate changes,
> one of
> them modifies channel.c:ast_channel_request_stream_topology_change() and
> the
> other modififies res_pjsip_session.c:resolve_refresh_media_states() (both
> very
> small in size).
>
> Since one of them touches the core and both probably require some
> discussion,
> would you prefer it if I openend separate reviews on Gerrit for them? Only
> in
> concert they fix ASTERISK-30184 but I am unsure of some unintended
> consequences
> since they're relatively major changes of the logic.
>

If they're fixing the same thing, then the same review is best in case it
is ultimately merged and needs to be reverted.

-- 
Joshua C. Colp
Asterisk Project Lead
Sangoma Technologies
Check us out at www.sangoma.com and www.asterisk.org
-- 
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

Reply via email to