[FFmpeg-devel] Reduce the size of AVRtpPayloadTypes

[Ronald S. Bultje]

Luca,

On Thu, Mar 27, 2008 at 7:06 AM, Luca Abeni <lucabe72 at email.it> wrote:

> -  {70, "unassigned", CODEC_TYPE_UNKNOWN, CODEC_ID_NONE, -1, -1},
> -  {71, "unassigned", CODEC_TYPE_UNKNOWN, CODEC_ID_NONE, -1, -1},
> -  {72, "reserved for RTCP conflict avoidance", CODEC_TYPE_UNKNOWN,
> CODEC_ID_NONE, -1, -1},
> -  {73, "reserved for RTCP conflict avoidance", CODEC_TYPE_UNKNOWN,
> CODEC_ID_NONE, -1, -1},
> -  {74, "reserved for RTCP conflict avoidance", CODEC_TYPE_UNKNOWN,
> CODEC_ID_NONE, -1, -1},
> -  {75, "reserved for RTCP conflict avoidance", CODEC_TYPE_UNKNOWN,
> CODEC_ID_NONE, -1, -1},
> -  {76, "reserved for RTCP conflict avoidance", CODEC_TYPE_UNKNOWN,
> CODEC_ID_NONE, -1, -1},
> -  {77, "unassigned", CODEC_TYPE_UNKNOWN, CODEC_ID_NONE, -1, -1},
> -  {78, "unassigned", CODEC_TYPE_UNKNOWN, CODEC_ID_NONE, -1, -1},
> [..]
> -  {114, "dynamic",   CODEC_TYPE_UNKNOWN, CODEC_ID_NONE, -1, -1},
> -  {115, "dynamic",   CODEC_TYPE_UNKNOWN, CODEC_ID_NONE, -1, -1},


Could you add comments /* 72-76 reserved for RTCP conflict avoidance */
newline /*77-etc. unassigned */ instead of completely removing them? Would
make it easier for those who don't have the specs in front of them to see
why they're missing.

Ronald