GitLab

Neither of those is guaranteed to be connected to framerate in any way
(if it even exists).

Fixes bug 56.

It's not supposed to be set outside of lavc.

compute_pkt_fields() is for unreliable estimates or guessing. The
keyframe information from the parser is (at least in theory) reliable,
so it should be used even when the other guessing is disabled with the
AVFMT_FLAG_NOFILLIN flag.

Therefore, move setting the packet keyframe flag based on parser
information from compute_pkt_fields() to read_frame_internal().

It is not supposed to be done outside lavc.

This is basically a revert of 818062f2.

It is unclear what issue this was supposed to fix, if it reappears again
it will have to be fixed in a more proper place.

The wtv-demux test change is because the sample starts with a B-frame.

This will allow parsers to export the duration of the current frame being
output, if known, instead of using AVCodecContext.frame_size.

Signed-off-by: Anton Khirnov <anton@khirnov.net>

Fixes bug 212.

This way, if the AVCodecContext is allocated for a specific codec, the
caller doesn't need to store this codec separately and then pass it
again to avcodec_open2().

It also allows to set codec private options using av_opt_set_* before
opening the codec.

It allows to check whether an AVCodecContext is open in a documented
way. Right now the undocumented way this check is done in lavf/lavc is
by checking whether AVCodecContext.codec is NULL. However it's desirable
to be able to set AVCodecContext.codec before avcodec_open2().

Give the exact aspect ratios when there is a mismatch between encoder
and muxer.
Signed-off-by: Anton Khirnov <anton@khirnov.net>

Also remove now unused AVFormatParameters struct and
AVOutputFormat.set_parameters().

If no data was seen for a stream decoder are returning 0 when fed with
empty packets for flushing. We can stop flushing when the decoder does
not return delayed delayed frames anymore. Changes try_decode_frame()
return value to got_picture or negative error.

CC: libav-stable@libav.org

Signed-off-by: Martin Storsjö <martin@martin.st>

The H.264 decoder needs SPS and PPS for initialization during
multi-threaded decoding. When probed single-threaded SPS and PPS are
copied to extradata and are available for proper initialization of
the decoder before the first frame is decoded.

Fixes a memleak.

This means that avformat_find_stream_info() now works properly with
multiple threads, so there's no need to force thread count to 1.

Signed-off-by: Martin Storsjö <martin@martin.st>

Fixes avformat_find_stream_info() on streams with number of frames <
thread count.

Fix the iformat/oformat typo.

It sets the supplied AVFormatContext pointer to NULL after freeing it,
which is safer and its name is consistent with other lavf functions.

Also deprecate av_close_input_file().

And remove all its uses.

libavformat/utils.c:2165:2: warning: label ‘fail’ defined but not used

It's supposed to be called only from (de)muxers.

If r_frame_rate is set, it should be more reliable for this than either
codec or stream timebase.

Adding the thread count in frame level multithreading to has_b_frames
as an additional delay causes more problems than it solves.
For example inconsistent behaviour during timestamp calculation in
libavformat.
Thread count and frame level multithreading are both set by the user.
If the additional delay caused by frame level multithreading needs
to be considered in the calling code it has all information to take
it into account.
Should it become necessary to calculate a maximum delay inside
libavcodec it should be exported as its own field and not reusing
an existing field.

Based on a patch by Michael Niedermayer.
Signed-off-by: Janne Grunau <janne-libav@jannau.net>

This makes the function accept the format of creation_time
as output by demuxers (e.g. the mov demuxer), making the
creation timestamp stay intact if transcoding.
Signed-off-by: Martin Storsjö <martin@martin.st>