- 26 Oct, 2015 4 commits
-
-
Michael Niedermayer authored
This fixes a fate failure after bumping the minor version Its unknown why this is not needed for the other aac tests, more investigation needed but for now i dont want to leave it broken while its investigated Signed-off-by:Michael Niedermayer <michael@niedermayer.cc>
-
Ganesh Ajjanagadde authored
filter_slice calls qsort, so qsort is in a performance critical position. AV_QSORT is substantially faster due to the inlining of the comparison callback. Thus, the increase in performance is worth the increase in binary size. Sample benchmark (x86-64, Haswell, GNU/Linux), filter-removegrain-mode-02 (from FATE) new: 24060 decicycles in qsort, 1 runs, 0 skips 15690 decicycles in qsort, 2 runs, 0 skips 9307 decicycles in qsort, 4 runs, 0 skips 5572 decicycles in qsort, 8 runs, 0 skips 3485 decicycles in qsort, 16 runs, 0 skips 2517 decicycles in qsort, 32 runs, 0 skips 1979 decicycles in qsort, 64 runs, 0 skips 1911 decicycles in qsort, 128 runs, 0 skips 1568 decicycles in qsort, 256 runs, 0 skips 1596 decicycles in qsort, 512 runs, 0 skips 1614 decicycles in qsort, 1024 runs, 0 skips 1874 decicycles in qsort, 2046 runs, 2 skips 2186 decicycles in qsort, 4094 runs, 2 skips old: 246960 decicycles in qsort, 1 runs, 0 skips 135765 decicycles in qsort, 2 runs, 0 skips 70920 decicycles in qsort, 4 runs, 0 skips 37710 decicycles in qsort, 8 runs, 0 skips 20831 decicycles in qsort, 16 runs, 0 skips 12225 decicycles in qsort, 32 runs, 0 skips 8083 decicycles in qsort, 64 runs, 0 skips 6270 decicycles in qsort, 128 runs, 0 skips 5321 decicycles in qsort, 256 runs, 0 skips 4860 decicycles in qsort, 512 runs, 0 skips 4424 decicycles in qsort, 1024 runs, 0 skips 4191 decicycles in qsort, 2046 runs, 2 skips 4934 decicycles in qsort, 4094 runs, 2 skips Reviewed-by:
Ganesh Ajjanagadde <gajjanagadde@gmail.com>
-
Paul B Mahol authored
Signed-off-by:Paul B Mahol <onemda@gmail.com>
-
Paul B Mahol authored
Signed-off-by:
-
- 25 Oct, 2015 15 commits
-
-
Ganesh Ajjanagadde authored
Reviewed-by:
-
Clément Bœsch authored
Example: % ./ffmpeg -h encoder=gif [...] GIF encoder AVOptions: -gifflags <flags> E..V.... set GIF flags (default offsetting+transdiff) offsetting E..V.... enable picture offsetting transdiff E..V.... enable transparency detection between frames -
Nicolas George authored
-
Ganesh Ajjanagadde authored
Commit 7c8fcbbd introduced some warnings that get triggered on the test build. This should fix them. Signed-off-by:
-
Carl Eugen Hoyos authored
Fixes ticket #4963.
-
Michael Niedermayer authored
Signed-off-by: -
Ganesh Ajjanagadde authored
ff_huff_build_tree uses qsort underneath. AV_QSORT is substantially faster due to the inlining of the comparison callback. Furthermore, this code is reasonably performance critical, since in e.g the fraps codec, ff_huff_build_tree is called on every frame. This routine is also called in vp6 on every frame in some circumstances. Sample benchmark (x86-64, Haswell, GNU/Linux), vp6 from FATE: vp6 (old): 78930 decicycles in qsort, 1 runs, 0 skips 45330 decicycles in qsort, 2 runs, 0 skips 27825 decicycles in qsort, 4 runs, 0 skips 17471 decicycles in qsort, 8 runs, 0 skips 12296 decicycles in qsort, 16 runs, 0 skips 9554 decicycles in qsort, 32 runs, 0 skips 8404 decicycles in qsort, 64 runs, 0 skips 7405 decicycles in qsort, 128 runs, 0 skips 6740 decicycles in qsort, 256 runs, 0 skips 7540 decicycles in qsort, 512 runs, 0 skips 9498 decicycles in qsort, 1024 runs, 0 skips 9938 decicycles in qsort, 2048 runs, 0 skips 8043 decicycles in qsort, 4095 runs, 1 skips vp6 (new): 15880 decicycles in qsort, 1 runs, 0 skips 10730 decicycles in qsort, 2 runs, 0 skips 10155 decicycles in qsort, 4 runs, 0 skips 7805 decicycles in qsort, 8 runs, 0 skips 6883 decicycles in qsort, 16 runs, 0 skips 6305 decicycles in qsort, 32 runs, 0 skips 5854 decicycles in qsort, 64 runs, 0 skips 5152 decicycles in qsort, 128 runs, 0 skips 4452 decicycles in qsort, 256 runs, 0 skips 4161 decicycles in qsort, 511 runs, 1 skips 4081 decicycles in qsort, 1023 runs, 1 skips 4072 decicycles in qsort, 2047 runs, 1 skips 4004 decicycles in qsort, 4095 runs, 1 skips Reviewed-by:
Timothy Gu <timothygu99@gmail.com> Reviewed-by:
-
Ganesh Ajjanagadde authored
Reviewed-by:
Ronald S. Bultje <rsbultje@gmail.com> Signed-off-by:
-
Ganesh Ajjanagadde authored
This adds const-correctness when needed for the comparators. Reviewed-by:
-
Paul B Mahol authored
Signed-off-by: -
Paul B Mahol authored
Signed-off-by: -
Paul B Mahol authored
Signed-off-by: -
Ganesh Ajjanagadde authored
libc's qsort comparator has a const qualifier on both arguments. This adds a missing const qualifier to exactly match the comparator API. Existing usages of av_tree_find, av_tree_insert are appropriately modified: type signature changes of the comparators, and removal of unnecessary void * casts of function pointers. Reviewed-by:
Henrik Gramner <henrik@gramner.com> Reviewed-by:
wm4 <nfxjfg@googlemail.com> Signed-off-by:
-
Ganesh Ajjanagadde authored
This documents the additional constness, and provides a useful libc reference for the API specification of the comparator. Reviewed-by:
-
Ganesh Ajjanagadde authored
For generality, qsort uses a comparator whose elements are void *. This makes the comparator have such a form, and thus makes the void * cast of the comparator pointer useless. Furthermore, this makes the code more consistent with other usages of qsort across the codebase. Reviewed-by:
-
- 24 Oct, 2015 10 commits
-
-
Ganesh Ajjanagadde authored
This fixes extra semicolons that clang 3.7 on GNU/Linux warns about. These were trigggered when built under -Wpedantic, which essentially checks for strict ISO compliance in numerous ways. Reviewed-by:
-
Ganesh Ajjanagadde authored
These casts are unnecessary, and may safely be removed. Found by enabling -Wpedantic on clang 3.7. Tested with FATE. Reviewed-by:
-
Ronald S. Bultje authored
Fixes ticket 4935.
-
Ronald S. Bultje authored
Fixes trac ticket 3226. Also see Andreas' analysis in https://bugs.debian.org/801745, which was very helpful.
-
Ganesh Ajjanagadde authored
ISO C requires at least one argument in the place of the ellipsis in a variadic macro. In particular, under -pedantic, this triggers the warning -Wgnu-zero-variadic-macro-arguments on clang 3.7. Reviewed-by:
Nicolas George <george@nsup.org> Signed-off-by:
-
Marton Balint authored
Only set pict if we got a valid rect. Signed-off-by:Marton Balint <cus@passwd.hu>
-
Marton Balint authored
Current code segfaults since the deprecation of AVSubtitleRect.pict because it freed/realloced AVSubtitleRect.pict.data by itself. The new code stores the generated YUVA AVSubtitle rectangles in their own struct and keeps the original AVSubtitle structure untouched, because overwriting it is considered invalid API usage. Signed-off-by: -
Carl Eugen Hoyos authored
-
Carl Eugen Hoyos authored
Reviewed-by: Tom Butterworth
-
Michael Niedermayer authored
avcodec/dpxenc: Fix "libavcodec/dpxenc.c:250:44: warning: passing argument 3 of av_image_copy_to_buffer from incompatible pointer type" Signed-off-by:
-
- 23 Oct, 2015 11 commits
-
-
Paul B Mahol authored
Our decoder have wrong name. Signed-off-by: -
Paul B Mahol authored
Signed-off-by: -
Paul B Mahol authored
Signed-off-by: -
Paul B Mahol authored
Signed-off-by: -
Paul B Mahol authored
Signed-off-by: -
Tom Butterworth authored
Signed-off-by: -
Ganesh Ajjanagadde authored
This reverts commit e6a93e59, Wstrict-prototypes is already enabled. Signed-off-by:
-
Ganesh Ajjanagadde authored
GCC (and Clang) have this useful warning that is not enabled by -Wall or -Wextra. This will ensure that issues like those fixed in 4da52e36 will trigger warnings. Reviewed-by:
Hendrik Leppkes <h.leppkes@gmail.com> Signed-off-by:
-
Ganesh Ajjanagadde authored
Reviewed-by:
Clément Bœsch <u@pkh.me> Signed-off-by:
-
Ronald S. Bultje authored
-
Tom Butterworth authored
fixes issue where alpha is ignored in some players Signed-off-by:
-
