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

I can't give any examples because in 10+ years I've never seen anyone use static payloads below 96 for dynamic codecs. That's my concern. It would not surprise me if some implementations don't handle it, or don't handle it well. We need to ensure that it is properly documented so that if someone experiences an issue they aren't stuck wondering what is going on and so we don't get flooded with issues about it. Could everything just be happy with it? Yes, but in my time with SIP and SDP I find that unlikely.

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