[FFmpeg-devel] [PATCH] H264 parser fate2 test

[Vitor Sessak]

On 07/09/2010 03:12 PM, M?ns Rullg?rd wrote:
> Vitor Sessak<vitor1001 at gmail.com>  writes:
>
>> $subj, as suggested in [1]. Sample is the first 1MB of the file in
>> issue944, now uploaded to [2] (will move to fate-suite/h264 is patch
>> ok). Test pass also in PPC.
>
> What exactly does this test?  Is it some strange corner-case not
> triggered by the standard test streams?

The standard test streams don't use the H264 parser, but you better look 
at the thread where the h264 gurus suggested that ([1]).

>> [1] http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/2009-April/022052.html
 
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

>> Index: tests/fate2.mak
>> ===================================================================
>> --- tests/fate2.mak	(revision 24138)
>> +++ tests/fate2.mak	(working copy)
>> @@ -2,3 +2,5 @@
>>   fate-twinvq: CMD = $(TARGET_PATH)/ffmpeg -i $(SAMPLES)/vqf/achterba.vqf -f s16le -
>>   fate-twinvq: CMP = oneoff
>>   fate-twinvq: REF = $(SAMPLES)/vqf/achterba.pcm
>> +FATE2_TESTS += fate-h264-parser
>> +fate-h264-parser: CMD = $$BUILD_PATH/ffmpeg -i $$SAMPLES_PATH/h264/h264_parser.mp4 -an -f framecrc -
>
> Please use $(TARGET_PATH) and $(SAMPLES) instead of the traditional
> fate variables.

Ok, new patch attached.

-Vitor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fate2_h264-parser2.diff
Type: text/x-patch
Size: 1322 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100709/e40c7be7/attachment.bin>