[FFmpeg-devel] [patch] mjpeg restart marker parsing

[Michael Niedermayer]

On Fri, Apr 02, 2010 at 09:35:10PM -0700, Tom Harper wrote:
> re:
> On 3/31/2010 3:23 AM, Michael Niedermayer wrote:
>> I would prefer it without the next_restart_marker related code.
>> if you want we could print all restart marker values if
>> debug&  FF_DEBUG_STARTCODE is set, but that should be a seperate patch
>>
>>    
> My test sample has restart markers every 10 macroblocks, any sort of 
> logging would not be very useful unless the restart markers were incorrect. 
>  At 720p/30fps such logging would render any other debugging info useless.  
> I don't see why the patch has to be re-written, given that it is 

> technically correct and informs you if the input is wrong?  Without the 
> next marker code you would never know this.

the error concealent code does this once its hooked up.
duplicating part of its functionalty like this is not ok

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100403/367d4df6/attachment.pgp>