[Asterisk-code-review] rtp: Add support for transport-cc in receiver direction. (...asterisk[16])

[Joshua Colp]

Joshua Colp has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/11315 )

Change subject: rtp: Add support for transport-cc in receiver direction.
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.asterisk.org/#/c/11315/1/res/res_rtp_asterisk.c 
File res/res_rtp_asterisk.c:

https://gerrit.asterisk.org/#/c/11315/1/res/res_rtp_asterisk.c@6600 
PS1, Line 6600: 
> Is the "instance" already locked or does the transport need to be locked, so there is not collision  […]
Everything is already locked. The RTP code does that on purpose so that deeper down (like in here) you don't have to worry about the locking unless you're trying to be clever.



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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I654a2cff5bd5554ab94457a14f70adb71f574afc
Gerrit-Change-Number: 11315
Gerrit-PatchSet: 1
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Comment-Date: Tue, 30 Apr 2019 20:27:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Kevin Harwell <kharwell at digium.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190430/91fac3d8/attachment.html>