[FFmpeg-devel] MPEG Video support in RTP

[Luca Barbato]

Luca Abeni wrote:
> Hi all,
> 
> ok, I have a patch for fixing the RTP packetization code for mpeg video. 
> It seems to work well (I tested it by receiving the video stream with 
> vlc), so I intend to submit it soon.

Great =)

> Of course I will split it in small patches, so since I've been asked to 
> move the codec-specific RTP code out of rtp.c I have some questions 
> about how to proceed:
> - Should this move of codec-dependent code be done as a first step?

Possibly yes.

> - If yes, should I create a new rtp_mpv.c file and move 
> rtp_send_mpegvideo() into it? (creating a file for a single function 
> seems overkilling)

I'd rather like to have it split, if everybody starts cluttering the
already ugly rtp.c with useful code we'll end up with an huge file...

> - Then, can I create rtp_mpv.h and include it in rtp.c, or is a more 
> complex method (similar to the one used for parsing dynamic protocols) 
> required?

inclusion looks fine for the time coming... (and we could make things
complex on a later time)

lu - there is already an almost generic mp4->rtp... we could hope

-- 

Luca Barbato

Gentoo/linux Gentoo/PPC
http://dev.gentoo.org/~lu_zero