[FFmpeg-devel] [PATCH] RTSP-MS 14/15: ASF packet parsing

Luca Abeni lucabe72
Tue Mar 24 13:09:11 CET 2009


Hi Ronald,

sorry, I seem to have missed some emails again...

Ronald S. Bultje wrote:
> Hi,
> 
> On Wed, Mar 18, 2009 at 7:29 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
>> On Tue, Feb 3, 2009 at 11:46 AM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
>>> Oops. I'll resubmit the patch prop'ed up.
>> Attached patch (sorry it took a while) fixes this.
>>
>> It also adds a new function ff_rtp_merge_data_packet() as suggested in
>> another thread (I forgot which one), because this mechanism is shared
>> by a whole lot of RTP payloads (e.g. X-QT, which I'll submit soon, I
>> hope). Basically this takes data from a RTP packet and appends it to a
>> buffer until a marker bit is set on a RTP packet, at which point it is
>> returned by the demuxer (this is the mechanism by which small UDP/RTP
>> packets can make up one big video frame, for example).
> 
> Ping? Luca (Abeni), could you review the merge function?

Where can I find the patch? (a link to ml archives would be ok).

I should not comment before reading the actual patch, but...
Judging from the comments above, I suspect this increases again the
confusion in the rtp demuxer: currently, the rtp demuxer is not supposed
to return a full frame, but uses a "parser" provided by libavcodec to
split the stream in frames. If I understand correctly what you wrote
above, your patch is changing this behaviour, but only for some payloads...


				Luca



More information about the ffmpeg-devel mailing list