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

Ronald S. Bultje rsbultje
Thu Jan 28 17:09:46 CET 2010


Hi,

On Tue, Jan 26, 2010 at 10:12 AM, Martin Storsj? <martin at martin.st> wrote:
> On Tue, 26 Jan 2010, Martin Storsj? wrote:
>> 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.)
>
> Actually attached this time...

Applied, thanks!

Ronald



More information about the ffmpeg-devel mailing list