[Ffmpeg-devel] Re: [PATCH] RTP/RTSP and MPEG4-AAC audio (Repost)

Romain DEGEZ romain.degez
Wed May 25 19:10:45 CEST 2005


Gary Corcoran wrote:
> Just one quick comment:
> RTP payload types 96 and up are *always* dynamic payload types.
> So why have all of them in a static table?  Why not just have
> the code check for payload type >= 96, and deal with the dynamic
> payload types that way?

You are right, these entries are useless but I let them for exhaustivity.
Whenever needed I'm checking payload_type against RTP_PT_PRIVATE (=96) to
handle dynamic payload type.

These entry could be removed without any change in the code.

> Just a suggestion,

Thanks :-)


Romain DEGEZ

More information about the ffmpeg-devel mailing list