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

[George Joseph]

George Joseph has posted comments on this change. ( https://gerrit.asterisk.org/3680 )

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


Patch Set 5: Code-Review-2

Alexander,  We've talked internally about this change and are feeling that re-using the static payload types might be problematic for peer devices that might not be able to handle using a type that is currently unasigned.  Josh has been working on an alternative solution that allows the dynamic payloads to be set on a per-session basis and we've opened a separate internal issue to address handling more than 32.  Given that, we'd like to close out this review.  If you're NOT ok with that, let us know in the next few days.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7bc96ab764bc30098a178b841cbf7146f9d64964
Gerrit-PatchSet: 5
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matthew Fredrickson <creslin at digium.com>
Gerrit-HasComments: No