[FFmpeg-devel] [PATCH] Realmedia / RTSP (RDT)

[Ronald S. Bultje]

Hi,

On Jan 2, 2008 11:04 AM, Michael Niedermayer <michaelni at gmx.at> wrote:

> On Tue, Jan 01, 2008 at 02:36:35PM -0500, Ronald S. Bultje wrote:
> > Index: ffmpeg/libavformat/rtp_internal.h
> > ===================================================================
> > --- ffmpeg.orig/libavformat/rtp_internal.h    2007-11-24 22:36:
> 33.000000000 -0500
> > +++ ffmpeg/libavformat/rtp_internal.h 2007-12-27 17:27:23.000000000-0500
> > @@ -46,7 +46,7 @@
> >                                                  AVPacket * pkt,
> >                                                  uint32_t *timestamp,
> >                                                  const uint8_t * buf,
> > -                                                int len);
> > +                                                int len, int flags);
> >
> >  typedef struct RTPDynamicProtocolHandler_s {
> >      // fields from AVRtpDynamicPayloadType_s
>
> could you please document per doxygen comment (in a .h file) what flags is
> good for (yes i really have forgotton, i just remember it was needed for
> subsequent patches)


New version with doxy comment for DynamicPayloadPachetHandlerProc attached.
In RDT, I use flags (in RTP_FLAG_*) to parse the "keyframe" flag from the
RTP packet header and forward it to the RDT packet parsing function and from
there to the rmdec.c packet parsing function (in normal .rm files, the flags
are in the first few bytes of each packet in the file, but those fields are
stripped in case of RDT because the RTP header already contains those fields
instead, so you have to read them from the RTP pachet header).

Ronald
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: rtsp-flags-field.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080104/c37b5619/attachment.asc>