On Thu, Sep 05, 2019 at 11:27:32AM -0700, Aman Gupta wrote:
> From: Aman Gupta <a...@tmm1.net>
>
> MediaCodec decoders require PTS for proper operation.
>
> Signed-off-by: Aman Gupta <a...@tmm1.net>
> ---
> libavcodec/mediacodecdec_common.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/libavcodec/mediacodecdec_common.c
> b/libavcodec/mediacodecdec_common.c
> index 1656cd6664..eae9c28d42 100644
> --- a/libavcodec/mediacodecdec_common.c
> +++ b/libavcodec/mediacodecdec_common.c
> @@ -612,7 +612,11 @@ int ff_mediacodec_dec_send(AVCodecContext *avctx,
> MediaCodecDecContext *s,
> }
>
> pts = pkt->pts;
> - if (pts != AV_NOPTS_VALUE && avctx->pkt_timebase.num &&
> avctx->pkt_timebase.den) {
> + if (pts == AV_NOPTS_VALUE) {
> + av_log(avctx, AV_LOG_WARNING, "Packet is missing PTS!\n");
Maybe reword to "Input packet is missing PTS" ? (without the !)
> + pts = 0;
> + }
> + if (pts && avctx->pkt_timebase.num && avctx->pkt_timebase.den) {
> pts = av_rescale_q(pts, avctx->pkt_timebase, AV_TIME_BASE_Q);
> }
>
> --
> 2.20.1
>
Except from my comment, LGTM.
--
Matthieu B.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
