[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