[Asterisk-code-review] rtp engine: Allow more than 32 dynamic payload types. (asterisk[13])

Alexander Traud asteriskteam at digium.com
Thu Aug 25 02:31:16 CDT 2016


Alexander Traud has posted comments on this change.

Change subject: rtp_engine: Allow more than 32 dynamic payload types.
......................................................................


Patch Set 1:

I was not about concrete implementations but example scenarios. or stated differently: I do not understand what you mean with 'do not handle it (well)'.

I have problems to extract any proposal from your comment 'properly documented': Do you want me to add a LOG_NOTICE to this code part, do you want me to add this to CHANGES, do you want me to add a LOG_NOTICE or VERBOSE whenever such a rtpmap is included/send by Asterisk in SIP/SDP?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7bc96ab764bc30098a178b841cbf7146f9d64964
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-HasComments: No



More information about the asterisk-code-review mailing list