[FFmpeg-devel] [PATCH] Set pix_fmt in svq3_decode_init

Mike Melanson mike
Sun Dec 13 16:48:51 CET 2009


M?ns Rullg?rd wrote:
> Mike Melanson <mike at multimedia.cx> writes:
> 
>> M?ns Rullg?rd wrote:
>>> Michael Niedermayer <michaelni at gmx.at> writes:
>>>
>>>> On Sat, Dec 12, 2009 at 04:49:46PM +0530, Jai Menon wrote:
>>>>> Hi,
>>>>>
>>>>> pix_fmt was earlier set in h264.c:decode_init but was later moved to
>>>>> h264.c:decode_slice_header. So currently, the svq3 decoder doesnt set
>>>>> avctx->pix_fmt (issue 1591 on roundup). Attached is a trivial patch to
>>>>> fix it but please correct me if i'm missing something.
>>>>>
>>>>> -- 
>>>>> Jai Menon
>>>>>
>>>>>  svq3.c |    1 +
>>>>>  1 file changed, 1 insertion(+)
>>>>> 73df4de9dbca537d210deb564768f9c82e920413  svq3_decode_fix.patch
>>>> Should be ok in principle, i have something similar in my tree, the
>>>> reason why i didnt commit it was that fate should be failing and it
>>>> seems it does not.
>>>> Once the fate issue is understood this can be commited of course
>>> The SVQ3 test is disabled because it was causing fate to fall over
>>> due
>>> to non-utf8 characters in the metadata printed from that file.  Now
>>> that we know this to be cause of those fate errors, that should be
>>> dealt with one way or another so the test can be turned on again.
>>> Where's Mike?
>> Always listening. Since SVQ3 is supposed to work again (or at least,
>> not make FATE fail in unusual ways), I have enabled the corresponding
>> test spec.
> 
> What did you do?

I flipped the switch from "don't test SVQ3" -> "test SVQ3". I see that 
icc 11.0 on x86_64 is back to having the same problem where it only 
inserts 127 tests but all the rest of the configs are fine.

-- 
     -Mike Melanson



More information about the ffmpeg-devel mailing list