Re: [FFmpeg-devel] [PATCH] ffmpeg.c: allow ffmpeg to output stats for each video stream - Make ffmpeg to output stats for each video/audio streams and each ouptut file ffmpeg output log in print_report. The report of video/audio sizes is clear now as previously all output video/audio sizes were combined to report and it is unclear such stats is for one output files or aggregates for all output files.

[Michael Niedermayer]

On Thu, Jun 14, 2018 at 08:35:48PM +0800, Wang Cao wrote:
> Signed-off-by: Wang Cao <wang...@google.com>
> ---
>  fftools/ffmpeg.c | 45 +++++++++++++++++++++++++++++++--------------
>  1 file changed, 31 insertions(+), 14 deletions(-)

this adds some warnings:

CC      fftools/ffmpeg.o
fftools/ffmpeg.c:1533:13: warning: function declaration isn’t a prototype 
[-Wstrict-prototypes]
 static void print_final_stats()
             ^
fftools/ffmpeg.c: In function ‘print_report’:
fftools/ffmpeg.c:1672:22: warning: unused variable ‘oc’ [-Wunused-variable]
     AVFormatContext *oc;
                      ^
                      
also the commit message should not be one very long line

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

What does censorship reveal? It reveals fear. -- Julian Assange

signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel