[FFmpeg-cvslog] avcodec/dca_core: move the reference in the bsf internal buffer

James Almer git at videolan.org
Sun Mar 11 20:47:22 EET 2018


ffmpeg | branch: master | James Almer <jamrial at gmail.com> | Sun Mar 11 13:29:19 2018 -0300| [aba437a6d0a7e0d9bafc6a2170ddd31d19d6c4d5] | committer: James Almer

avcodec/dca_core: move the reference in the bsf internal buffer

There's no need to allocate a new packet for it.

Reviewed-by: Mark Thompson <sw at jkqxz.net>
Signed-off-by: James Almer <jamrial at gmail.com>

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

 libavcodec/dca_core_bsf.c | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/libavcodec/dca_core_bsf.c b/libavcodec/dca_core_bsf.c
index 9edc0cfd61..8565796951 100644
--- a/libavcodec/dca_core_bsf.c
+++ b/libavcodec/dca_core_bsf.c
@@ -24,18 +24,17 @@
 #include "dca_syncwords.h"
 #include "libavutil/mem.h"
 
-static int dca_core_filter(AVBSFContext *ctx, AVPacket *out)
+static int dca_core_filter(AVBSFContext *ctx, AVPacket *pkt)
 {
-    AVPacket *in;
     GetByteContext gb;
     uint32_t syncword;
     int core_size = 0, ret;
 
-    ret = ff_bsf_get_packet(ctx, &in);
+    ret = ff_bsf_get_packet_ref(ctx, pkt);
     if (ret < 0)
         return ret;
 
-    bytestream2_init(&gb, in->data, in->size);
+    bytestream2_init(&gb, pkt->data, pkt->size);
     syncword = bytestream2_get_be32(&gb);
     bytestream2_skip(&gb, 1);
 
@@ -45,11 +44,8 @@ static int dca_core_filter(AVBSFContext *ctx, AVPacket *out)
         break;
     }
 
-    av_packet_move_ref(out, in);
-    av_packet_free(&in);
-
-    if (core_size > 0 && core_size <= out->size) {
-        out->size = core_size;
+    if (core_size > 0 && core_size <= pkt->size) {
+        pkt->size = core_size;
     }
 
     return 0;



More information about the ffmpeg-cvslog mailing list