[FFmpeg-devel] [PATCH 5/9] ffmpeg: move video stats code to do_video_out().

[Clément Bœsch]

From: Clément Bœsch <clement.boesch at smartjog.com>

This will allow some simplifications in transcode_video().
---
 ffmpeg.c |   15 +++++++--------
 1 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/ffmpeg.c b/ffmpeg.c
index 623e5ca..35874ad 100644
--- a/ffmpeg.c
+++ b/ffmpeg.c
@@ -1446,18 +1446,20 @@ static void do_video_resample(OutputStream *ost,
 #endif
 }
 
+static void do_video_stats(AVFormatContext *os, OutputStream *ost, int frame_size);
 
 static void do_video_out(AVFormatContext *s,
                          OutputStream *ost,
                          InputStream *ist,
                          AVFrame *in_picture,
-                         int *frame_size, float quality)
+                         float quality)
 {
     int nb_frames, i, ret, format_video_sync;
     AVFrame *final_picture;
     AVCodecContext *enc;
     double sync_ipts;
     double duration = 0;
+    int frame_size = 0;
 
     enc = ost->st->codec;
 
@@ -1474,8 +1476,6 @@ static void do_video_out(AVFormatContext *s,
     /* by default, we output a single frame */
     nb_frames = 1;
 
-    *frame_size = 0;
-
     format_video_sync = video_sync_method;
     if (format_video_sync == VSYNC_AUTO)
         format_video_sync = (s->oformat->flags & AVFMT_VARIABLE_FPS) ? ((s->oformat->flags & AVFMT_NOTIMESTAMPS) ? VSYNC_PASSTHROUGH : VSYNC_VFR) : 1;
@@ -1572,7 +1572,7 @@ static void do_video_out(AVFormatContext *s,
                 if (format_video_sync == VSYNC_DROP)
                     pkt.pts = pkt.dts = AV_NOPTS_VALUE;
                 write_frame(s, &pkt, ost);
-                *frame_size = ret;
+                frame_size = ret;
                 video_size += ret;
 
                 /* if two pass, output log */
@@ -1589,6 +1589,8 @@ static void do_video_out(AVFormatContext *s,
          */
         ost->frame_number++;
     }
+    if (vstats_filename && frame_size)
+        do_video_stats(output_files[ost->file_index].ctx, ost, frame_size);
 }
 
 static double psnr(double d)
@@ -2124,7 +2126,6 @@ static int transcode_video(InputStream *ist, AVPacket *pkt, int *got_output, int
     for (i = 0; i < nb_output_streams; i++) {
         AVFrame *filtered_frame = NULL;
         OutputStream *ost = &output_streams[i];
-        int frame_size;
 
         if (!check_output_constraints(ist, ost) || !ost->encoding_needed)
             continue;
@@ -2153,10 +2154,8 @@ static int transcode_video(InputStream *ist, AVPacket *pkt, int *got_output, int
             filtered_frame = decoded_frame;
 #endif
 
-            do_video_out(output_files[ost->file_index].ctx, ost, ist, filtered_frame, &frame_size,
+            do_video_out(output_files[ost->file_index].ctx, ost, ist, filtered_frame,
                          same_quant ? quality : ost->st->codec->global_quality);
-            if (vstats_filename && frame_size)
-                do_video_stats(output_files[ost->file_index].ctx, ost, frame_size);
 #if CONFIG_AVFILTER
             cont:
             avfilter_unref_buffer(ost->picref);
-- 
1.7.8.3