| Commit message (Collapse) | Author | Age |
|
|
|
| |
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
|
| |
|
|
|
|
|
|
| |
The sdl_supported_color_spaces only meant for SDL builtin renderer.
Signed-off-by: Zhao Zhili <zhilizhao@tencent.com>
|
|
|
|
|
|
|
| |
Broken in a2fc86378a18b2c2966ce3438df8f27f646438e5.
Reviewed-by: Anton Khirnov <anton@khirnov.net>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
|
|
|
|
|
|
| |
This makes ffprobe match the rest of the codebase.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
|
|
|
|
| |
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
|
|
|
|
|
|
| |
Allows to avoid casting const away.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
|
|
|
|
| |
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
|
|
|
|
|
|
|
| |
It can be triggered when send_to_enc_thread() returns AVERROR(ENOMEM).
Propagate the error to the caller instead.
Reported-by: Andreas Rheinhardt
|
| |
|
| |
|
|
|
|
| |
It causes write_option() to return 0 when calling func_arg() fails.
|
| |
|
| |
|
|
|
|
|
|
|
| |
Rather than read it directly from InputStream.
This is a step towards avoiding the assumption that filtergraph inputs
are always fed by demuxers.
|
|
|
|
|
|
|
| |
Rather than read them directly from InputStream.
This is a step towards avoiding the assumption that filtergraph inputs
are always fed by demuxers.
|
|
|
|
|
|
|
| |
Rather than read them directly from InputStream.
This is a step towards avoiding the assumption that filtergraph inputs
are always fed by demuxers.
|
|
|
|
|
|
|
| |
Do not construct the name manually from input file/stream indices.
This is a step towards avoiding the assumption that filtergraph inputs
are always fed by demuxers.
|
|
|
|
|
|
| |
The computation is based on demuxer properties, so that is the more
appropriate place for it. Filter code just receives the desired
start time/duration.
|
|
|
|
|
| |
It should never be necessary now that decoders propagate global side
data to frames.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The filename is freed with the OptionsContext and therefore
there will be a use-after-free when reporting the filename
in print_stream_maps(). So create a copy of the string.
This is a regression since 8aed3911fc454e79697e183660bf30d31334a64b.
fate-lavf-mkv_attachment exhibits it (and reports a random nonsense
filename here), but this does not make the test fail (not even with
valgrind; only with ASAN, as it aborts on use-after-free).
Reviewed-by: Anton Khirnov <anton@khirnov.net>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Makes it robust against adding fields before it, which will be useful in
following commits.
Majority of the patch generated by the following Coccinelle script:
@@
typedef AVOption;
identifier arr_name;
initializer list il;
initializer list[8] il1;
expression tail;
@@
AVOption arr_name[] = { il, { il1,
- tail
+ .unit = tail
}, ... };
with some manual changes, as the script:
* has trouble with options defined inside macros
* sometimes does not handle options under an #else branch
* sometimes swallows whitespace
|
|
|
|
|
|
|
|
|
|
|
|
| |
SDL supports only these three matrices. Actually, it only supports these
three combinations: BT.601+JPEG, BT.601+MPEG, BT.709+MPEG, but we have
no way to restrict the specific *combination* of YUV range and YUV
colorspace with the current filter design.
See-Also: https://trac.ffmpeg.org/ticket/10839
Instead of an incorrect conversion result, trying to play a YCgCo file
with ffplay will simply error out with a "No conversion possible" error.
|
|
|
|
|
| |
This avoids assuming that it survives unchanged through bitstream
filters, if present.
|
|
|
|
|
| |
Reviewed-by: Anton Khirnov <anton@khirnov.net>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
|
|
|
|
| |
Reported-by: Andreas Rheinhardt
|
| |
|
|
|
|
|
|
| |
Fix bug #10501
Signed-off-by: WHR <msl0000023508@gmail.com>
|
|
|
|
| |
There's no need to go through full string expansion here.
|
|
|
|
|
|
|
|
|
| |
This commit lets ffplay properly propagate YUV metadata into the filter
graph, avoiding such issues as e.g. accidentally passing YCgCo into a
filter that can't support it. Also fixes an error related to this
missing metadata from buffersrc (since commit 2d555dc82d)
See-Also: https://trac.ffmpeg.org/ticket/10839
|
| |
|
|
|
|
| |
It is no longer used outside of ffmpeg_demux.
|
|
|
|
| |
It is no longer used outside of ffmpeg_demux.
|
|
|
|
|
|
|
|
|
|
|
| |
Previously, the demuxer would register decoder with the scheduler, using
InputStream as opaque, and pass the scheduling index to the decoder.
Now the registration is done by the decoder itself, using DecoderPriv as
opaque, and the scheduling index is returned to demuxer from dec_open().
decoder_thread() then no longer needs to be accessed from outside of
ffmpeg_dec and can be made static.
|
| |
|
| |
|
|
|
|
|
|
| |
Do not retrieve it from InputStream directly.
This is a step towards decoupling Decoder and InputStream.
|
|
|
|
|
|
| |
Do not retrieve it from InputStream directly.
This is a step towards decoupling Decoder and InputStream.
|
|
|
|
|
|
| |
The same information can also be obtained from the decoder itself.
This is a step towards decoupling Decoder and InputStream.
|
|
|
|
|
|
| |
Do not build it from InputStream values.
This is a step towards decoupling Decoder and InputStream.
|
|
|
|
|
|
| |
Do not read it from InputStream directly.
This is a step towards decoupling Decoder and InputStream.
|
|
|
|
|
|
| |
Do not read it from AVStream directly.
This is a step towards decoupling Decoder and InputStream.
|
|
|
|
|
|
|
| |
It is done based on demuxer information, so that is the more appropriate
place for this code.
This is a step towards decoupling Decoder and InputStream.
|
|
|
|
|
|
| |
Stop reading them from InputStream.
This is a step towards decoupling Decoder and InputStream.
|
|
|
|
| |
Will be useful in the following commit.
|
|
|
|
|
|
| |
Stop reading them from InputStream.
This is a step towards decoupling Decoder and InputStream.
|
|
|
|
|
| |
This function is decoding-only and has no interaction with the rest of
ffmpeg_hw. It thus belongs more properly in ffmpeg_dec.
|
|
|
|
|
|
| |
Reduces the need to access InputFile from decoding.
This is a step towards decoupling Decoder and InputStream.
|
|
|
|
|
|
|
| |
* as this decision is based on demuxing information, move it from the
decoder to the demuxer
* as the issue being addressed is latency added by frame threading, we
only need to disable frame threading, not all threading
|
|
|
|
|
|
| |
Pass this information to dec_open() instead.
This is a step towards decoupling Decoder and InputStream.
|