avcodec/mediacodecdec: add delay_flush option
authorAman Gupta <aman@tmm1.net>
Tue, 6 Mar 2018 21:07:32 +0000 (13:07 -0800)
committerAman Gupta <aman@tmm1.net>
Thu, 8 Mar 2018 00:22:47 +0000 (16:22 -0800)
commit2a0eb8685728ccb260e42f60e1dbefe47ababbc3
treef84f97754d63a1bbf77baccc9520e5b8d24dff41
parent840f6eb77aed6c9dc2eff1a89add3611ec305262
avcodec/mediacodecdec: add delay_flush option

The default behavior of the mediacodec decoder before this commit
was to delay flushes until all pending hardware frames were
returned to the decoder. This was useful for certain types of
applications, but was unexpected behavior for others.

The new default behavior with this commit is now to execute
flushes immediately to invalidate all pending frames. The old
behavior can be enabled by setting delay_flush=1.

With the new behavior, video players implementing seek can simply
call flush on the decoder without having to worry about whether
they have one or more mediacodec frames still buffered in their
rendering pipeline. Previously, all these frames had to be
explictly freed (or rendered) before the seek/flush would execute.

The new behavior matches the behavior of all other lavc decoders,
reducing the amount of special casing required when using the
mediacodec decoder.

Signed-off-by: Aman Gupta <aman@tmm1.net>
Signed-off-by: Matthieu Bouron <matthieu.bouron@gmail.com>
doc/APIchanges
libavcodec/mediacodec.c
libavcodec/mediacodecdec.c
libavcodec/mediacodecdec_common.c
libavcodec/mediacodecdec_common.h
libavcodec/version.h