[FFmpeg-devel] [RFC] RTP headers cleanup

Luca Barbato lu_zero
Fri Jan 16 10:42:40 CET 2009


Luca Abeni wrote:
> Hi Luca,
> 
> Luca Barbato wrote:
> [...]
>> +++ b/libavformat/rtpenc_h264.h
>> that looks a bit overkill
> 
> Yes, I had the same impression... But then I do not know where to put
> the prototypes of all the ff_rtp_send_*() functions... Would it be
> ok to put those prototypes in rtpenc.h? (in this way, I can also remove
> rtp_mpv.h and rtp_aac.h). Or should I add a list of "extern ff_rtp_send_*()"
> in rtpenc.c? (I do not like the idea, but I see something similar is done
> in allformats.c, for example).

I'd factor all of them in a single header file like rtp_send.h or 
rtp_payloader.h

> Yes, I was thinking about splitting all the dynamic payload stuff in a
> different header file, for example. But I think that can be done in a
> second batch of patches, if people agree.

ok

-- 

Luca Barbato
Gentoo Council Member
Gentoo/linux Gentoo/PPC
http://dev.gentoo.org/~lu_zero





More information about the ffmpeg-devel mailing list