[FFmpeg-devel] [PATCH] flvdec start-of-frame fix
Thu Apr 1 03:09:13 CEST 2010
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
>>>>> always read from the start of a frame, instead of reading from the end
>>>>> 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...
More information about the ffmpeg-devel