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

Martin Storsjö martin
Tue Jan 26 16:09:37 CET 2010


On Tue, 26 Jan 2010, Ronald S. Bultje wrote:

> I actually meant to index them all, now you read the memory twice. I.e.:
> 
> +    flags          =  AV_RB16(buf);
> +    startcode      = (flags & 0x0400) >> 17;
> +    vrc            =  flags & 0x02;
> +    picture_header = (flags & 0x01f8) >> 3;
> 
> But that's nitpicking, either way is fine with me. If you prefer
> current behaviour, then this is OK also. Patch OK.

Updated patch attached, that does things this way. (Named the variable 
"header" instead of "flags", since there's already a parameter named 
flags.)

// Martin



More information about the ffmpeg-devel mailing list