[FFmpeg-devel] [PATCH] flvdec start-of-frame fix

[Howard Chu]

Michael Niedermayer wrote:
> On Wed, Mar 31, 2010 at 04:49:48PM -0700, Howard Chu wrote:
>> Michael Niedermayer wrote:
>>> On Tue, Mar 30, 2010 at 06:42:50PM -0700, Howard Chu wrote:
>>>>> Note that the did_rseek flag wouldn't even be needed if the rest of
>>>>> flvdec
>>>>> always read from the start of a frame, instead of reading from the end
>>>>> of
>>>>> the previous frame, as it currently does.
>>
>>> putting the url_fskip() inside the loop at the end seems cleaner to me
>>>
>>> also the url_fskip() movement could be a seperate patch
>>
>> OK, patch separated, url_fskip() moved to inside the loop.
>
> i did not realize the continue problem, thatt said the patch looks
> ok assuming the regression tests pass.
> If not you could try for(;;url_fskip)
> which would avoid the sligtly ugly ,1 and should work with continue
> pure nitpicking i know ...

Yeah, moving into the for-iterator is fine. "make test" succeeds.

-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: flvdif.txt
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100331/3ba0155f/attachment.txt>