omx: Don't return > 0 from omx_encode_frame
authorMartin Storsjö <martin@martin.st>
Thu, 23 Jun 2016 22:13:16 +0000 (01:13 +0300)
committerMartin Storsjö <martin@martin.st>
Fri, 24 Jun 2016 19:30:54 +0000 (22:30 +0300)
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@martin.st>
libavcodec/omx.c

index 63c7f5b..0c61c2f 100644 (file)
@@ -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