[FFmpeg-devel] [PATCH] fix sdp(file) vs rtsp dep
Luca Abeni
lucabe72
Thu Nov 12 10:20:22 CET 2009
Diego Biurrun wrote:
> On Wed, Nov 11, 2009 at 11:28:24AM +0100, Luca Abeni wrote:
>> Luca Abeni wrote:
>> [...]
>>>> Anyway, the patch is quite big. Maybe you could split it by first
>>>> moving the functions around.
>>> Ok; I'll try to do this when I'll find some more time to work on this
>>> patch.
>> Next round of patches:
>> - move_functions-2.diff moves some code around, to simplify the following
>> patches
>
> Luca B told me on IRC that moving functions around is OK, so you should
> just commit this.
Ok; I am going to commit it soon.
>> - split_read-1.diff splits the sdp_read_packet() function out of
>> rtsp_read_packet(). This allows to avoid compiling RTSP code when not
>> needed (as done in the next patch).
>> - add-ifdefs.diff adds some ifdefs to avoid compiling the RTSP code when
>> the RTSP demuxer is disabled, and removes a useless "#if
>> CONFIG_SDP_DEMUXER"
>> The last patch should make it clear why I think that the RTSP demuxer
>> should depend on the SDP demuxer and not vice-versa...
>>
>> I did not do any renaming yet, but I think it can be done later.
>> Some work is still needed to split the code in two different files
>> (sdp.c and rtsp.c) but I have no time right now...
>
> I'm not sure if this is really simpler than renaming and splitting the
> file right away, whatever, if you think it is, go for it...
It surely is faster... And I have not much time right now :)
I'll commit this unless anyone disagrees.
>> --- ffmpeg.orig/libavformat/rtsp.c 2009-11-11 09:11:15.000000000 +0100
>> +++ ffmpeg/libavformat/rtsp.c 2009-11-11 09:12:48.000000000 +0100
>> @@ -43,6 +43,8 @@
>> //#define DEBUG
>> //#define DEBUG_RTP_TCP
>>
>> +static int tcp_read_packet(AVFormatContext *s, RTSPStream **prtsp_st,
>> + uint8_t *buf, int buf_size);
>> static int rtsp_read_reply(AVFormatContext *s, RTSPMessageHeader *reply,
>> unsigned char **content_ptr,
>> int return_on_interleaved_data);
>
> Ugh, more forward declarations. Can't these be avoided?
this one (tcp_read_packet()) can be avoided by moving the tcp_read_packet()
function (which is used only by RTSP, hence is under "#if CONFIG_RTSP_DEMUXER")
before sdp_read_packet(). So, this would increase the number of "#if"s.
If this is ok, I can remove the forward declaration for tcp_read_packet().
The forward declaration for rtsp_read_reply() could be avoided by removing
its invocation from udp_read_packet() (and moving it to rtsp_read_packet()).
But this would be a functional change that can potentially break something,
so I'd prefer to have it in a different commit. (I still have to try such
an idea, so I have no patch yet).
Luca
More information about the ffmpeg-devel
mailing list