Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

to make dca_decode_frame more readable
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

And avoid a memory leak.
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

packets are queued due to packet reordering until the queue reach its
maximal size or max delay is reached.
This commit adds a warning trace when max delay is reached.
Signed-off-by: NEloi BAIL <eloi.bail@savoirfairelinux.com>
Signed-off-by: NMartin Storsjö <martin@martin.st>

This commit print as AV_LOG_VERBOSE the jitter buffer
size. It might be the default value or the value set by application.
Signed-off-by: NEloi BAIL <eloi.bail@savoirfairelinux.com>
Signed-off-by: NMartin Storsjö <martin@martin.st>

This commit adds a warning trace when jitter buffer
is full. It helps to understand leading decoding issues.
Signed-off-by: NEloi BAIL <eloi.bail@savoirfairelinux.com>
Signed-off-by: NMartin Storsjö <martin@martin.st>

Since the actual max length of the jitter buffer is restricted by
max_delay, this shouldn't harm the overall latency (assuming that
max_delay is set properly), while allowing packet reordering with
a larger number of packets (which may be required with high bitrate
video).
Signed-off-by: NMartin Storsjö <martin@martin.st>

Makes debugging a little simpler.

Move the packet-specific parsers in separate functions.

That loop is completely stand-alone.

Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Add necessary headers in .c files.

MAX_MB_BYTES -> mpegutils.h
FRAME_SKIPPED -> mpegutils.h
INPLACE_OFFSET -> mpegutils.h
ME_MAP_SIZE -> motion_est.h

Bug-Id: CID 1323180

Add timebase and aspect ratio information.

"int" is useful in testing because provides accurate results across
different plaftforms, so remove it from the scheduled FF_API_UNUSED_MEMBERS
deprecation.

This introduces a slight timebase computation difference in zmbv-8bit
fate test. This is expected since the new options are double instead
of ints, and the additional precision skews the results in a non
meaningful way.

Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Fix typos, add missing comments, and remove stray lines.
Signed-off-by: NVittorio Giovara <vittorio.giovara@gmail.com>

Deprecate the now unused option, but temporarily retain the capability
to disable the now default behaviour.

Mention this change in the AVPacket documentation.
Signed-off-by: NVittorio Giovara <vittorio.giovara@gmail.com>

bc hasn't been needed since a982c5d7Signed-off-by: NJames Almer <jamrial@gmail.com>
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

This also drops setting the frame->pts field. This is usually not set by
decoders, so this would be an inconsistency that's at worst a danger to
the API user.

It appears the buffer->dts field is normally not set by the MMAL
decoder, so don't use it. If it's ever going to be set by MMAL, we
don't know whether the value will be what we want.
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

The generic code in utils.c sets the AVFrame.pkt_dts field from the
packet it was supposedly decoded. This does not have to be true for a
fully asynchronous decoder like mmaldec. It could be overwritten with an
incorrect value. Even if the decoder doesn't determine the DTS (but sets
it to AV_NOPTS_VALUE), it's impossible to determine a correct value in
utils.c.

Decoders can now be marked with FF_CODEC_CAP_SETS_PKT_DTS, in which case
utils.c won't overwrite the field. The decoders are expected to set this
field (even if they only set it to AV_NOPTS_VALUE).
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

This MMAL feature fills in missing timestamps from the framerate set on
the input port. This is generally unwanted, since libavcodec decoders
merely pass through timestamps without ever "fixing" them. The framerate
is also unknown, and even the timebase doesn't have to be set.
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Don't try to do a blocking wait for MMAL output if we haven't even sent
a single real packet, but only flush packets. Obviously we can't expect
to get anything back.

Additionally, don't send a flush packet to MMAL in the same case. It
appears the MMAL decoder will sometimes hang in mmal_vc_port_disable()
(called from ffmmal_close_decoder()), waiting for a reply from the GPU
which never arrives. Either MMAL disallows sending flush packets without
preceding real data, or it's a MMAL bug.
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

I can't come up with a nice way to handle this. It's hard to keep the
lock-stepped input/output in this case. You can't predict whether the
MMAL decoder will output a picture (because it's asynchronous), so
you have to assume in general that any packet could produce 0 or 1
frames. You can't continue to write input packets to the decoder,
because then you might get too many output frames, which you can't
get rid of because the lavc decoding API does not allow the decoder
to return an output frame without consuming an input frame (except
when flushing).

The ideal fix is a M:N decoding API (preferably asynchronous), which
would make this code potentially much cleaner. For now, this hack
will do.
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

That loop is completely stand-alone.

Matches the openjpeg behaviour.

Bug-Id: 887
CC: libav-stable@libav.org

Bug-Id: 886
CC: libav-stable@libav.org