[FFmpeg-cvslog] avcodec/vp9: Inline function with only one caller into it

Andreas Rheinhardt git at videolan.org
Tue May 11 01:20:52 EEST 2021


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Wed May  5 21:27:21 2021 +0200| [4818ea69d2dab5e96e7c2c078d804a260d5efcbd] | committer: Andreas Rheinhardt

avcodec/vp9: Inline function with only one caller into it

The split into vp9_decode_init() and init_frames() is a remnant
of using init_thread_copy() for frame-threading; the latter has
been removed, so there is no reason for init_frames() not be part
of vp9_decode_init().

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>

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

 libavcodec/vp9.c | 25 ++++++++-----------------
 1 file changed, 8 insertions(+), 17 deletions(-)

diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c
index fd0e0dc54f..783f446125 100644
--- a/libavcodec/vp9.c
+++ b/libavcodec/vp9.c
@@ -1790,36 +1790,27 @@ static void vp9_decode_flush(AVCodecContext *avctx)
         ff_thread_release_buffer(avctx, &s->s.refs[i]);
 }
 
-static int init_frames(AVCodecContext *avctx)
+static av_cold int vp9_decode_init(AVCodecContext *avctx)
 {
     VP9Context *s = avctx->priv_data;
-    int i;
 
-    for (i = 0; i < 3; i++) {
+    s->last_bpp = 0;
+    s->s.h.filter.sharpness = -1;
+
+    for (int i = 0; i < 3; i++) {
         s->s.frames[i].tf.f = av_frame_alloc();
         if (!s->s.frames[i].tf.f)
             return AVERROR(ENOMEM);
     }
-    for (i = 0; i < 8; i++) {
-        s->s.refs[i].f = av_frame_alloc();
-        s->next_refs[i].f = av_frame_alloc();
+    for (int i = 0; i < 8; i++) {
+        s->s.refs[i].f      = av_frame_alloc();
+        s->next_refs[i].f   = av_frame_alloc();
         if (!s->s.refs[i].f || !s->next_refs[i].f)
             return AVERROR(ENOMEM);
     }
-
     return 0;
 }
 
-static av_cold int vp9_decode_init(AVCodecContext *avctx)
-{
-    VP9Context *s = avctx->priv_data;
-
-    s->last_bpp = 0;
-    s->s.h.filter.sharpness = -1;
-
-    return init_frames(avctx);
-}
-
 #if HAVE_THREADS
 static int vp9_decode_update_thread_context(AVCodecContext *dst, const AVCodecContext *src)
 {



More information about the ffmpeg-cvslog mailing list