[Asterisk-code-review] chan_pjsip: fix music on hold continues after INVITE with replaces (asterisk[18])

N A asteriskteam at digium.com
Sat Apr 1 17:19:07 CDT 2023


Attention is currently required from: Henning Westerholt.

N A has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/20020 )

Change subject: chan_pjsip: fix music on hold continues after INVITE with replaces
......................................................................


Patch Set 1: Code-Review-1

(2 comments)

Commit Message:

https://gerrit.asterisk.org/c/asterisk/+/20020/comment/a89cc435_62554c73 
PS1, Line 10: continue 
continues


File res/res_pjsip_refer.c:

https://gerrit.asterisk.org/c/asterisk/+/20020/comment/b1df456b_d05fcdb5 
PS1, Line 1022: 	/* Unhold the channel already, as later we are not having access to it anymore */
How about rephrasing as "Unhold the channel now, as later we don't have access to it anymore".

I think that's a bit clearer.



-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/20020
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 18
Gerrit-Change-Id: I5675df11e739be5226b328f8828d4b8d81fbefb4
Gerrit-Change-Number: 20020
Gerrit-PatchSet: 1
Gerrit-Owner: Henning Westerholt <hw at gilawa.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: N A <asterisk at phreaknet.org>
Gerrit-Attention: Henning Westerholt <hw at gilawa.com>
Gerrit-Comment-Date: Sat, 01 Apr 2023 22:19:07 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20230401/d47e8c70/attachment.html>


More information about the asterisk-code-review mailing list