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

[Romain DEGEZ]

Hi,

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

Regards

-- 
Romain DEGEZ