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

[Joshua Colp]

Joshua Colp has posted comments on this change.

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


Patch Set 1:

You are correct that they shouldn't be and for new codecs it's probably fine. My concern is for implementations that treat anything below 96 as static and don't support the MAY from the RFC. If they gracefully handle things or we never offer, then great. If we do and something breaks - that's my concern.

-- 
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