[FFmpeg-devel] [PATCH] RTP depacketizer for H.263

[Reimar Döffinger]

On Mon, Jan 25, 2010 at 05:09:21PM +0100, Michael Niedermayer wrote:
> On Mon, Jan 25, 2010 at 05:05:23PM +0200, Martin Storsj? wrote:
> [...]
> > +static int h263_handle_packet(AVFormatContext *ctx,
> > +                              PayloadContext *data,
> > +                              AVStream *st,
> > +                              AVPacket * pkt,
> > +                              uint32_t * timestamp,
> > +                              const uint8_t * buf,
> > +                              int len, int flags)
> > +{
> > +    uint8_t *ptr;
> > +    int P, V, plen, pebit;
> > +
> > +    if (len < 2) {
> > +        av_log(ctx, AV_LOG_ERROR, "Too short H.263 RTP packet\n");
> > +        return AVERROR_INVALIDDATA;
> > +    }
> > +
> > +    /* Decode the 16 bit H.263+ payload header, as described in section
> > +     * 5.1 of RFC 4629.
> > +     * If P is set, it indicates a picture start, picture segment start
> > +     * or video sequence end, and two zero bytes should be prepended to
> > +     * the payload.
> > +     * If V is set, an 8 bit Video Redundancy Coding field is present
> > +     * after this 16 bit header.
> > +     * plen is the length (in bytes) of an extra picture header, following
> > +     * the VRC field.
> > +     * pebit is the number of bits to ignore of the last byte of the extra
> > +     * picture header.
> > +     */
> 
> N d p v n. A F t v n f
> 
> A // Always
> d // document
> F // fix
> f // first
> N // Never
> n // names
> p // poor
> t // the
> v // variable

I think we need that in the quotes selection if not in the "developer manual" :-)