[Asterisk-code-review] rtp engine: allocate RTP dynamic payloads per session (asterisk[14])

Mark Michelson asteriskteam at digium.com
Wed Mar 22 10:15:39 CDT 2017


Mark Michelson has posted comments on this change. ( https://gerrit.asterisk.org/5259 )

Change subject: rtp_engine: allocate RTP dynamic payloads per session
......................................................................


Patch Set 1: Code-Review-1

(2 comments)

https://gerrit.asterisk.org/#/c/5259/1/main/rtp_engine.c
File main/rtp_engine.c:

PS1, Line 1309:  * \note Both static_RTP_PT_lock and codecs (if given) must be at least
              :  *       least read locked before calling.
"at least least"


PS1, Line 1445:                /*
              :                 * The payload type is a static assignment
              :                 * or our default dynamic position is available.
              :                 */
              :               
Looks like you've indented this with spaces instead of tabs.


-- 
To view, visit https://gerrit.asterisk.org/5259
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If282877199b85da8dde7eb9452cdedaa19da586a
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 14
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list