[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 michael at niedermayer.cc
Fri Jun 15 21:51:55 EEST 2018


On Thu, Jun 14, 2018 at 08:35:48PM +0800, Wang Cao wrote:
> Signed-off-by: Wang Cao <wangcao at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180615/dcf64b40/attachment.sig>


More information about the ffmpeg-devel mailing list