| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
Entries are organized in descending chronological order.
Signed-off-by: Rostislav Pehlivanov <[email protected]>
|
|
|
|
|
|
| |
Long overdue for removal, af_aresample should be used instead.
Signed-off-by: Rostislav Pehlivanov <[email protected]>
|
|
|
|
|
|
|
|
|
| |
Could lead to random behavior. This possibly happened due to commit
32a5b631267. This should/could probably be simplified, but for no apply
a minimal fix to quell the errors.
Tested-by: Michael Niedermayer <[email protected]>
Reviewed-by: Michael Niedermayer <[email protected]>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
async_mutex has is used in a very strange but intentional way: it is
locked by default, and unlocked only in regions that can be run
concurrently.
If the user was calling API functions to the same context from different
threads (in a safe way), this could unintentionally unlock the mutex on
a different thread than the previous lock operation. It's not allowed by
the pthread API.
Fix this by emulating a binary semaphore using a mutex and condition
variable. (Posix semaphores are not available on all platforms.)
Tested-by: Michael Niedermayer <[email protected]>
Reviewed-by: Michael Niedermayer <[email protected]>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The old "API" that signaled rotation as a metadata value has been
replaced by DISPLAYMATRIX side data quite a while ago.
There is no reason to make muxers/demuxers/API users support both. In
addition, the metadata API is dangerous, as user tags could "leak" into
it, creating unintended features or bugs.
ffmpeg CLI has to be updated to use the new API. In particular, we must
not allow to leak the "rotate" tag into the muxer. Some muxers will
catch this properly (like mov), but others (like mkv) can add it as
generic tag. Note applications, which use libavformat and assume the
old rotate API, will interpret such "rotate" user tags as rotate
metadata (which it is not), and incorrectly rotate the video.
The ffmpeg/ffplay tools drop the use of the old API for muxing and
demuxing, as all muxers/demuxers support the new API. This will mean
that the tools will not mistakenly interpret per-track "rotate" user
tags as rotate metadata. It will _not_ be treated as regression.
Unfortunately, hacks have been added, that allow the user to override
rotation by setting metadata explicitly, e.g. via
-metadata:s:v:0 rotate=0
See references to trac #4560. fate-filter-meta-4560-rotate0 tests this.
It's easier to adjust the hack for supporting it than arguing for its
removal, so ffmpeg CLI now explicitly catches this case, and essentially
replaces the "rotate" value with a display matrix side data. (It would
be easier for both user and implementation to create an explicit option
for rotation.)
When the code under FF_API_OLD_ROTATE_API is disabled, one FATE
reference file has to be updated (because "rotate" is not exported
anymore).
Tested-by: Michael Niedermayer <[email protected]>
Reviewed-by: Michael Niedermayer <[email protected]>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Squelches the following compiler warnings:
libavcodec/opusenc.c:1051:16: warning: format specifies type 'long' but
the argument has type 'long long' [-Wformat]
avctx->bit_rate/1000, clipped_rate/1000);
^~~~~~~~~~~~~~~~~~~~
libavcodec/opusenc.c:1051:38: warning: format specifies type 'long' but
the argument has type 'long long' [-Wformat]
avctx->bit_rate/1000, clipped_rate/1000);
^~~~~~~~~~~~~~~~~
|
|
|
|
|
| |
Reviewed-by: "Ronald S. Bultje" <[email protected]>
Signed-off-by: Michael Niedermayer <[email protected]>
|
|
|
|
|
|
|
|
| |
This was skipped in c17563c5d3c974a69709ebae0171534763b3051c because
it depended on the filter setup merge, but was forgotten after that
actually happened.
Fixes hwaccel fate for stream size change tests.
|
|
|
|
| |
Signed-off-by: Michael Niedermayer <[email protected]>
|
|
|
|
|
|
|
| |
Fixes: 712/clusterfuzz-testcase-6647676227551232
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg
Signed-off-by: Michael Niedermayer <[email protected]>
|
|
|
|
| |
Signed-off-by: Michael Niedermayer <[email protected]>
|
|
|
|
| |
Signed-off-by: Michael Niedermayer <[email protected]>
|
|
|
|
|
| |
Reviewed-by: Rostislav Pehlivanov <[email protected]>
Signed-off-by: James Almer <[email protected]>
|
|
|
|
|
|
|
| |
It's a simplifaction of the same code, originally commited as 3b5ad8fbf77.
Reviewed-by: Michael Niedermayer <[email protected]>
Signed-off-by: James Almer <[email protected]>
|
|
|
|
|
|
|
| |
It's a simplifaction of the same code, originally commited as b4b9a64bdb6.
Reviewed-by: Michael Niedermayer <[email protected]>
Signed-off-by: James Almer <[email protected]>
|
|
|
|
|
|
|
|
|
| |
Changes to the parsing code originally committed to mpegvideo_parser.c
in 73fb23dc5a5.
Required by some samples, like PVA_test-partial.pva
Reviewed-by: Michael Niedermayer <[email protected]>
Signed-off-by: James Almer <[email protected]>
|
|
|
|
| |
Signed-off-by: James Almer <[email protected]>
|
|
|
|
| |
Signed-off-by: James Almer <[email protected]>
|
|
|
|
| |
Signed-off-by: James Almer <[email protected]>
|
|
|
|
| |
Signed-off-by: James Almer <[email protected]>
|
|
|
|
| |
around
|
| |
|
| |
|
|
|
|
|
|
|
| |
The first case was forgotten in 89790ba2bfc9d0dc5ad407c5724b6ee616ecde58.
Note: build_only=yes is one of the case where hiding the errors makes
sense.
|
|
|
|
| |
libavcodec/xface.c:318:27: warning: assuming signed overflow does not occur when assuming that (X - c) > X is always false [-Wstrict-overflow]
|
|
|
|
|
|
| |
Fixes empty statement, found by ubitux
Signed-off-by: Michael Niedermayer <[email protected]>
|
|
|
|
|
| |
Signed-off-by: Thomas Turner <[email protected]>
Signed-off-by: Michael Niedermayer <[email protected]>
|
|
|
|
|
|
| |
Fixes #6214 and vsynth1-dnxhd-720p-hr-lb.
Signed-off-by: Paul B Mahol <[email protected]>
|
|
|
|
|
|
|
| |
3d6535983282bea542dac2e568ae50da5796be34
Unrolling the loops triplicates the size of the assembled output
while not generating any gain in performance.
|
|\
| |
| |
| |
| |
| |
| |
| |
| | |
* commit 'fbe425c8d29e473a8f69ae2dc52b1a10b77f3b44':
hap: Adjust printf length modifiers to match variable types
This commit is a noop, see 5a51ca2da7b76cad2a86476590c18b26a98eafbe
Merged-by: Clément Bœsch <[email protected]>
|
| |
| |
| |
| |
| | |
libavcodec/hapenc.c:121:20: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘size_t {aka unsigned int}’ [-Wformat=]
libavcodec/hapenc.c:121:20: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 5 has type ‘size_t {aka unsigned int}’ [-Wformat=]
|
|\|
| |
| |
| |
| |
| |
| | |
* commit '1263b2039eb5aaf1522e9de9f07c787ab30a5f50':
Adjust printf conversion specifiers to match variable signedness
Merged-by: Clément Bœsch <[email protected]>
|
| | |
|
|\|
| |
| |
| |
| |
| |
| | |
* commit 'ca1e5eea0c7b72a6e30aa6488cfeced3a4853521':
Remove some pointless TRACE level debug code
Merged-by: Clément Bœsch <[email protected]>
|
| |
| |
| |
| | |
This also kills some warnings with certain compiler options.
|
|\|
| |
| |
| |
| |
| |
| | |
* commit '07eea5a5ded1141632aefecfa59dcdc26de2d7ea':
nut: Drop pointless TRACE level debug code
Merged-by: Clément Bœsch <[email protected]>
|
| |
| |
| |
| | |
The code has little usefulness and uses the __PRETTY_FUNCTION__ GNU extension.
|
|\|
| |
| |
| |
| |
| |
| | |
* commit 'c3dad1bf3b5e04e01c291b1ac41e6bef0adf2206':
nsv: Drop unnecessary TRACE level debug code
Merged-by: Clément Bœsch <[email protected]>
|
| |
| |
| |
| | |
The output is rather silly and the code uses non-standard __FUNCTION__.
|
|\|
| |
| |
| |
| |
| |
| | |
* commit '47756f51fe836959ffa5c6e2baeacbd71e150069':
dnxhdenc: Drop pointless, commented-out debug output
Merged-by: Clément Bœsch <[email protected]>
|
| | |
|
|\|
| |
| |
| |
| |
| |
| | |
* commit '0456e684394dc5a7b98ab9ebb48396d743bf3730':
audio_fifo: Drop write-only variable
Merged-by: Clément Bœsch <[email protected]>
|
| | |
|
|\|
| |
| |
| |
| |
| |
| | |
* commit '0574780d7a196f87ddd89d6362f4c47f3532b4c4':
h264_loopfilter: Do not print value of uninitialized variable
Merged-by: Clément Bœsch <[email protected]>
|
| |
| |
| |
| | |
libavcodec/h264_loopfilter.c:531:111: warning: variable 'edge' is uninitialized when used here [-Wuninitialized]
|
|\|
| |
| |
| |
| |
| |
| | |
* commit '255526998501f0040ae43fe4848c817a97fc578a':
mpegaudio: Do not print value of uninitialized variable
Merged-by: Clément Bœsch <[email protected]>
|
| |
| |
| |
| | |
libavcodec/mpegaudiodec_template.c:885:97: warning: variable 'x' is uninitialized when used here [-Wuninitialized]
|
|\|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* commit '14cab426b03afd08bc9fe9b6e021a9543c4bdd7e':
build: Hardcode avversion.h dependency
This commit is a noop, see:
4d0ac9aee920cd95c35fbf4cd0a073baf86ff4c2
4ae87554f3c8bc54db572873f5049427a7e6cb31
Merged-by: Clément Bœsch <[email protected]>
|
| |
| |
| |
| |
| |
| |
| | |
Since avversion.h is a generated header it must be created before
dependencies can be determined as a side effect of compilation.
Otherwise Make stops and restarts the build process to generate
avversion.h and produces related error messages.
|
|\|
| |
| |
| |
| |
| |
| |
| |
| | |
* commit 'f22363c72968f1a1fc4881d8695ec7068b0aa03c':
openssl: Avoid double semicolons after the GET_BIO_DATA macro
This commit is a noop, see fc83de7e1d0eab29942b4ceb67b49e66e7b99226
Merged-by: Clément Bœsch <[email protected]>
|