[FFmpeg-devel] [PATCH] tests/fate:Add FATE for hls_flags append option

Steven Liu lingjiujianke at gmail.com
Wed Aug 24 16:40:55 EEST 2016


2016-08-24 19:14 GMT+08:00 Steven Liu <lingjiujianke at gmail.com>:

>
>
> 2016-08-24 18:57 GMT+08:00 Michael Niedermayer <michael at niedermayer.cc>:
>
>> On Wed, Aug 24, 2016 at 06:28:08PM +0800, Steven Liu wrote:
>> > 2016-08-24 18:23 GMT+08:00 Michael Niedermayer <michael at niedermayer.cc
>> >:
>> >
>> > > On Wed, Aug 24, 2016 at 03:04:59PM +0800, Steven Liu wrote:
>> > > > On OSX test:
>> > > > localhost:ffmpeg liuqi$ uname -a
>> > > > Darwin localhost 15.5.0 Darwin Kernel Version 15.5.0: Tue Apr 19
>> 18:36:36
>> > > > PDT 2016; root:xnu-3248.50.21~8/RELEASE_X86_64 x86_64
>> > > > localhost:ffmpeg liuqi$ make fate-filter-hls-append
>> > > > HOSTCC tests/base64.o
>> > > > HOSTLD tests/base64
>> > > > HOSTCC tests/tiny_psnr.o
>> > > > HOSTLD tests/tiny_psnr
>> > > > HOSTCC tests/tiny_ssim.o
>> > > > HOSTLD tests/tiny_ssim
>> > > > HOSTCC tests/audiomatch.o
>> > > > tests/audiomatch.c:102:12: warning: using floating point absolute
>> value
>> > > > function 'fabs' when argument is of integer type [-Wabsolute-value]
>> > > >         if(fabs(c) > sigamp * 0.94)
>> > > >            ^
>> > > > tests/audiomatch.c:102:12: note: use function 'llabs' instead
>> > > >         if(fabs(c) > sigamp * 0.94)
>> > > >            ^~~~
>> > > >            llabs
>> > > > tests/audiomatch.c:103:40: warning: using floating point absolute
>> value
>> > > > function 'fabs' when argument is of integer type [-Wabsolute-value]
>> > > >             maxshift = FFMIN(maxshift, fabs(pos)+32);
>> > > >                                        ^
>> > > > tests/audiomatch.c:103:40: note: use function 'abs' instead
>> > > >             maxshift = FFMIN(maxshift, fabs(pos)+32);
>> > > >                                        ^~~~
>> > > >                                        abs
>> > > > tests/audiomatch.c:25:28: note: expanded from macro 'FFMIN'
>> > > > #define FFMIN(a,b) ((a) > (b) ? (b) : (a))
>> > > >                            ^
>> > > > tests/audiomatch.c:103:40: warning: using floating point absolute
>> value
>> > > > function 'fabs' when argument is of integer type [-Wabsolute-value]
>> > > >             maxshift = FFMIN(maxshift, fabs(pos)+32);
>> > > >                                        ^
>> > > > tests/audiomatch.c:103:40: note: use function 'abs' instead
>> > > >             maxshift = FFMIN(maxshift, fabs(pos)+32);
>> > > >                                        ^~~~
>> > > >                                        abs
>> > > > tests/audiomatch.c:25:34: note: expanded from macro 'FFMIN'
>> > > > #define FFMIN(a,b) ((a) > (b) ? (b) : (a))
>> > > >                                  ^
>> > > > tests/audiomatch.c:104:12: warning: using floating point absolute
>> value
>> > > > function 'fabs' when argument is of integer type [-Wabsolute-value]
>> > > >         if(fabs(c)>fabs(bestc)){
>> > > >            ^
>> > > > tests/audiomatch.c:104:12: note: use function 'llabs' instead
>> > > >         if(fabs(c)>fabs(bestc)){
>> > > >            ^~~~
>> > > >            llabs
>> > > > 4 warnings generated.
>> > > > HOSTLD tests/audiomatch
>> > > > GEN     tests/data/hls-list-append.m3u8
>> > > > TEST    filter-hls-append
>> > > >
>> > > >
>> > > >
>> > > >
>> > > > On Linux Test:
>> > > > [root at localhost ffmpeg]# make fate-filter-hls-append
>> > > > HOSTCC tests/base64.o
>> > > > HOSTLD tests/base64
>> > > > HOSTCC tests/tiny_psnr.o
>> > > > HOSTLD tests/tiny_psnr
>> > > > HOSTCC tests/tiny_ssim.o
>> > > > HOSTLD tests/tiny_ssim
>> > > > HOSTCC tests/audiomatch.o
>> > > > tests/audiomatch.c: 在函数‘main’中:
>> > > > tests/audiomatch.c:40: 警告:此函数中的‘bestpos’在使用前可能未初始化
>> > > > HOSTLD tests/audiomatch
>> > > > GEN     tests/data/hls-list-append.m3u8
>> > > > TEST    filter-hls-append
>> > > > [root at localhost ffmpeg]# uname -a
>> > > > Linux localhost.localdomain 2.6.32-431.el6.x86_64 #1 SMP Fri Nov 22
>> > > > 03:15:09 UTC 2013 x86_64 x86_64 x86_64 GNU/Linux
>> > >
>> > > fails with mingw
>> > > ../configure  --cc='ccache i686-w64-mingw32-gcc'  --samples=...
>> --arch=x86
>> > > --target-os=mingw32 --cross-prefix=i686-w64-mingw32- --enable-gpl
>> > > --pkg-config=./pig-config --target_exec=wine
>> > >
>> > > ...
>> > > +0,     873216,     873216,     1152,     2304, 0x823b822c
>> > > +0,     874368,     874368,     1152,     2304, 0xa1658479
>> > > +0,     875520,     875520,     1152,     2304, 0xdbe58ff7
>> > > +0,     876672,     876672,     1152,     2304, 0x725175e2
>> > > +0,     877824,     877824,     1152,     2304, 0xb2ae7741
>> > > +0,     878976,     878976,     1152,     2304, 0x4de169e4
>> > > +0,     880128,     880128,     1152,     2304, 0x3cb18530
>> > > +0,     881280,     881280,     1152,     2304, 0x48b95d6c
>> > > +0,     882432,     882432,     1152,     2304, 0xcff707ef
>> > > Test filter-hls-append failed. Look at tests/data/fate/filter-hls-app
>> end.err
>> > > for details.
>> > > make: *** [fate-filter-hls-append] Error 1
>> > > make: *** Waiting for unfinished jobs....
>> > >
>> > ...... ok, let me find a Windows Environment to fix it.
>>
>> the failure was on a ubuntu with wine, i dont know if it fails on a
>> native windows
>> ok, i see, i'll test this patch on windows
>
>
Hi Michael,

      I have check the patch on Windows System, It's OK,

Message:

Administrator at DELLDEL-A6K6BMH ~/ffmpeg/xxx
$ make fate-filter-hls-append
HOSTCC  tests/base64.o
HOSTLD  tests/base64.exe
HOSTCC  tests/tiny_psnr.o
HOSTLD  tests/tiny_psnr.exe
HOSTCC  tests/tiny_ssim.o
HOSTLD  tests/tiny_ssim.exe
HOSTCC  tests/audiomatch.o
C:/MinGW/msys/1.0/home/Administrator/ffmpeg/tests/audiomatch.c: In function
'mai
n':
C:/MinGW/msys/1.0/home/Administrator/ffmpeg/tests/audiomatch.c:109:5:
warning: '
bestpos' may be used uninitialized in this function [-Wmaybe-uninitialized]
     printf("presig: %d postsig:%d c:%7.4f lenerr:%d\n", bestpos, datlen -
sigle
n - bestpos, bestc / sigamp, datlen - siglen);
     ^
HOSTLD  tests/audiomatch.exe
GEN     tests/data/hls-list-append.m3u8
TEST    filter-hls-append

Administrator at DELLDEL-A6K6BMH ~/ffmpeg/xxx
$ uname -a
MINGW32_NT-6.1 DELLDEL-A6K6BMH 1.0.19(0.48/3/2) 2016-07-13 17:45 i686 Msys

Administrator at DELLDEL-A6K6BMH ~/ffmpeg/xxx
$



And I will send a new path for fix the warning when complie audiomatch.c


More information about the ffmpeg-devel mailing list