| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
| |
This fixes https://bugzilla.libav.org/show_bug.cgi?id=514.
Signed-off-by: Martin Storsjö <martin@martin.st>
|
|
|
|
|
|
|
| |
Notwithstanding the return value the frame should be freed if
is not going to be used.
Reported-by: Matthew Wolenetz <wolenetz@chromium.org>
|
| |
|
| |
|
|
|
|
|
| |
Otherwise some code might mistakenly think it is allowed to unref those
buffers, which would lead to double unref.
|
|
|
|
| |
Needed e.g. for h264 cropping to work properly.
|
| |
|
|
|
|
| |
Signed-off-by: Martin Storsjö <martin@martin.st>
|
|
|
|
|
|
|
| |
Since c977039e585bfff28ecc037ef827c6c3d1ed88aa plane count for
PIX_FMT_HWACCEL pixel formats is 0 instead of 1. The created dummy
AVBuffers are still bogus since AVFrame does not hold frame data when
AVHWAccels are used.
|
|
|
|
|
|
|
| |
Also move the declaration to internal.h, and add restrict qualifiers
to the declaration (as in the implementation).
Signed-off-by: Martin Storsjö <martin@martin.st>
|
|
|
|
|
|
|
|
|
|
| |
This allows dropping the mpegvideo dependency from a number of
components.
This also fixes standalone building of the h264 parser, which
was broken in 64e438697.
Signed-off-by: Martin Storsjö <martin@martin.st>
|
|
|
|
|
|
| |
Number of planes is not always equal to the number of components even
for formats marked with PIX_FMT_PLANAR -- e.g. NV12 has three components
in two planes.
|
|
|
|
|
| |
It is supposed to be height * linesize, not width * linesize.
Thanks to Hendrik Leppkes for pointing out the bug.
|
|
|
|
|
| |
This allows reporting missing features and requesting samples from
all libraries in a standard way; with a simplified API.
|
|
|
|
|
|
| |
AVCodecContext release_buffer() shall be NULL for audio codecs using
get_buffer. The backward compatibility code hence have to check before
calling it.
|
|
|
|
|
| |
planes - FF_ARRAY_ELEMS would be evaluated as unsigned and underflow
instead of being negative as was intended.
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Fixes build without threads after
759001c534287a96dc96d1e274665feb7059145d.
|
|
|
|
|
| |
This should ensure that a valid channel layout is always set on a frame,
until a better solution is implemented.
|
| |
|
| |
|
|
|
|
|
|
| |
This will allow us to avoid copying the packets in many cases.
This breaks ABI.
|
|
|
|
|
|
| |
Make sure the behavior does not change with the locale.
Signed-off-by: Martin Storsjö <martin@martin.st>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Make av_get_codec_tag_string() show codec tag string characters in a more
intelligible ways. For example the ascii char "@" is used as a number, so
should be displayed like "[64]" rather than as a printable character.
Apart alphanumeric chars, only the characters ' ' and '.' are used
literally in codec tags, all the other characters represent numbers.
This also avoids relying on locale-dependent character class functions.
Signed-off-by: Martin Storsjö <martin@martin.st>
|
|
|
|
|
| |
There may be more decoders that rely on this. Those should be found and
fixed.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
rc_buffer_size is not set before.
Solve the initial the rate control underflow issue reported in
bug 222.
CC: libav-stable@libav.org
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
|
|
|
|
|
| |
libavcodec/utils.c:1050:5: warning: missing braces around initializer [-Wmissing-braces]
libavcodec/utils.c:1314:5: warning: missing braces around initializer [-Wmissing-braces]
|
|
|
|
|
|
|
|
| |
Direct rendering capable decoders call get_buffer() which will set the
frame parameters.
Prevents frames with wrong parameters when a decoder outputs delayed
frames after a resolution or pixel format change.
|
| |
|
|
|
|
|
| |
Any performance gain from this is negligible and not worth the extra
code.
|
|
|
|
|
| |
There is no point in duplicating this code in every get_buffer()
implementation.
|
|
|
|
| |
It will be useful in the upcoming transition to refcounted AVFrames.
|
|
|
|
|
| |
It is used only in one place and is unlikely it would be needed
elsewhere.
|
|
|
|
| |
Deprecate avcodec_get_chroma_sub_sample.
|
|
|
|
| |
Also reorder some other #include when applicable.
|
|
|
|
|
|
| |
Same fix and issue as in a25d912dca9cd553440167e0476c47581359c0fc
Signed-off-by: Anton Khirnov <anton@khirnov.net>
|
|
|
|
|
|
| |
Ensures the decoder did not set channel count to an insanely high value
during initialization, which could cause large memory usage when it tries to
get a buffer during decoding.
|
| |
|
|
|
|
|
|
|
| |
Avoids memleaks with audio when extended_data is nontrivial and the user
doesn't reset the frame.
Shouldn't have any effect for video for now, but might be useful in the
future.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Error out on init if a codec with CODEC_CAP_EXPERIMENTAL is requested
and strict_std_compliance is not FF_COMPLIANCE_EXPERIMENTAL.
Move the check from avconv to avcodec_open2() and return
AVERROR_EXPERIMENTAL accordingly.
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
|
|
|
|
| |
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
|
|
|
|
|
|
|
| |
Since 2bc0de385, AVFrame needs to be initialized
before calling avcodec_get_frame_defaults().
Signed-off-by: Anton Khirnov <anton@khirnov.net>
|
|
|
|
| |
Functions used across libraries should have avpriv_ and not ff_ prefixes.
|
| |
|
| |
|