[FFmpeg-devel] [PATCH] ffmpeg: print data size too in statistic

Michael Niedermayer michaelni at gmx.at
Fri Jan 31 01:44:05 CET 2014


On Fri, Jan 31, 2014 at 12:45:40AM +0100, Stefano Sabatini wrote:
> On date Thursday 2014-01-30 20:35:37 +0100, Michael Niedermayer encoded:
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> >  ffmpeg.c |    3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/ffmpeg.c b/ffmpeg.c
> > index 5aeb961..fc53b1d 100644
> > --- a/ffmpeg.c
> > +++ b/ffmpeg.c
> > @@ -1316,10 +1316,11 @@ static void print_report(int is_last_report, int64_t timer_start, int64_t cur_ti
> >      if (is_last_report) {
> >          int64_t raw= audio_size + video_size + data_size + subtitle_size + extra_size;
> >          av_log(NULL, AV_LOG_INFO, "\n");
> > -        av_log(NULL, AV_LOG_INFO, "video:%1.0fkB audio:%1.0fkB subtitle:%1.0f global headers:%1.0fkB muxing overhead %f%%\n",
> > +        av_log(NULL, AV_LOG_INFO, "video:%1.0fkB audio:%1.0fkB subtitle:%1.0f data:%1.0f global headers:%1.0fkB muxing overhead %f%%\n",
> >                 video_size / 1024.0,
> >                 audio_size / 1024.0,
> >                 subtitle_size / 1024.0,
> > +               data_size / 1024.0,
> >                 extra_size / 1024.0,
> >                 100.0 * (total_size - raw) / raw
> >          );
> 
> Why not?

wasnt sure because it makes the line longer and the additional
information is probably 0 in most cases so not so usefull

but
applied

i dont mind to revert if people find it was better before

also i could make it ommit cases that are 0 but then that might annoy
anyone who tries to parse it with a script

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

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140131/8a4ceab0/attachment.asc>


More information about the ffmpeg-devel mailing list