[Asterisk-code-review] codecs: Add Codec 2 mode 2400. (asterisk[master])

[Alexander Traud]

Alexander Traud has posted comments on this change.

Change subject: codecs: Add Codec 2 mode 2400.
......................................................................


Patch Set 6:

Related only indirectly to this issue here, however, because Codec 2 or SILK is going to get 120 by using -1 after this is included, there was a reason why 120 is free: <https://github.com/asterisk/asterisk/commit/7ba7ec9c976b44ba7cdf21b7c7cb8ab8aaf20b44> at least there was in the year 2003 because iLBC got moved to 97 back then.

Yet, I was not able to find an issue report or another comment related to that change. Can somebody ask Mark? Some Cisco phones seem to use 120 for DTMF but in Asterisk the payload type 121 was appointed to that. I am confused.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1e45d8084683fab5f2b272bf35f4a149cea8b8d6
Gerrit-PatchSet: 6
Gerrit-Project: asterisk
Gerrit-Branch: master
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-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matthew Fredrickson <creslin at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-HasComments: No