[FFmpeg-devel] [PATCH] rtsp.c small cleanups

Ronald S. Bultje rsbultje
Sat Mar 21 22:07:43 CET 2009


Hi,

On Sat, Mar 21, 2009 at 9:52 AM, Luca Barbato <lu_zero at gentoo.org> wrote:
[..]
> Ok

All applied, except:

>> (8) remove sdp_read_packet() and replace it with rtsp_read_packet()
>> because that's all it does
>
> rtsp_read_packet should be splitted at least in 3...

OK, so we just discussed this, it indeed seems like it's not as straightforward:
1) rtsp_read_packet() is getting a bit bloated and should probably be
split into a few subfunctions
2) sdp_read_packet() might not have to call all of these

So I'll be working on that separately outside this thread.

Ronald



More information about the ffmpeg-devel mailing list