[FFmpeg-cvslog] omx: Don't return > 0 from omx_encode_frame

Martin Storsjö git at videolan.org
Fri Oct 7 14:28:59 EEST 2016


ffmpeg | branch: master | Martin Storsjö <martin at martin.st> | Fri Jun 24 01:13:16 2016 +0300| [0c9c4004ed57de210b4d83c7b39bbfb00b86b9af] | committer: Martin Storsjö

omx: Don't return > 0 from omx_encode_frame

The encode function is supposed to just return 0 on success.
This stems from a mixup with the return value of decode functions.

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

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=0c9c4004ed57de210b4d83c7b39bbfb00b86b9af
---

 libavcodec/omx.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/libavcodec/omx.c b/libavcodec/omx.c
index 63c7f5b..0c61c2f 100644
--- a/libavcodec/omx.c
+++ b/libavcodec/omx.c
@@ -845,7 +845,6 @@ static int omx_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
                 s->output_buf_size = 0;
             }
             if (buffer->nFlags & OMX_BUFFERFLAG_ENDOFFRAME) {
-                ret = pkt->size;
                 pkt->pts = av_rescale_q(from_omx_ticks(buffer->nTimeStamp), AV_TIME_BASE_Q, avctx->time_base);
                 // We don't currently enable B-frames for the encoders, so set
                 // pkt->dts = pkt->pts. (The calling code behaves worse if the encoder



More information about the ffmpeg-cvslog mailing list