[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