[FFmpeg-devel] [PATCH] Set pix_fmt in svq3_decode_init
Sun Dec 13 04:00:55 CET 2009
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:
>>> 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.
More information about the ffmpeg-devel