Re: [FFmpeg-devel] [PATCH] Add option to log timing

2019-09-06 Thread Soft Works
> From: ffmpeg-devel On Behalf Of > Michael Niedermayer > Sent: Friday, September 6, 2019 8:56 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH] Add option to log timing > > On Wed, Sep 04, 2019 at 07:35:11PM

Re: [FFmpeg-devel] [PATCH] Add option to log timing

2019-09-06 Thread Michael Niedermayer
On Wed, Sep 04, 2019 at 07:35:11PM +, Soft Works wrote: > > > Why this restriction? I think all log lines should be start with > > time/date if corresponding flag is present. This makes the log to be > > easy to parse by scripts. > > Initially I didn’t have this restriction, but it doesn’t wo

Re: [FFmpeg-devel] [PATCH] Add option to log timing

2019-09-04 Thread Soft Works
> Michael Niedermayer wrote: > > > > > Why this restriction? I think all log lines should be start with > > > time/date if corresponding flag is present. This makes the log to be > > > easy to parse by scripts. > > > > Initially I didn’t have this restriction, but it doesn’t work well together >

[FFmpeg-devel] [PATCH] Add option to log timing

2019-09-04 Thread Soft Works
0001-Add-option-to-log-timing.patch Description: 0001-Add-option-to-log-timing.patch ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ..

Re: [FFmpeg-devel] [PATCH] Add option to log timing

2019-09-04 Thread Michael Niedermayer
On Wed, Sep 04, 2019 at 07:35:11PM +, Soft Works wrote: > > > Why this restriction? I think all log lines should be start with > > time/date if corresponding flag is present. This makes the log to be > > easy to parse by scripts. > > Initially I didn’t have this restriction, but it doesn’t wo

Re: [FFmpeg-devel] [PATCH] Add option to log timing

2019-09-04 Thread Michael Niedermayer
On Wed, Sep 04, 2019 at 06:53:28PM +, Soft Works wrote: > This commit adds two logging flags: 'timing' and 'datetiming'. > > Usage: ffmpeg -logleve +timing > > Timing information will only be added to log lines once transcoding has > started. > --- > fftools/cmdutils.c | 14 ++ >

Re: [FFmpeg-devel] [PATCH] Add option to log timing

2019-09-04 Thread Soft Works
> Why this restriction? I think all log lines should be start with > time/date if corresponding flag is present. This makes the log to be > easy to parse by scripts. Initially I didn’t have this restriction, but it doesn’t work well together with some multi-line logging. See below for an example

Re: [FFmpeg-devel] [PATCH] Add option to log timing

2019-09-04 Thread Bodecs Bela
Hi, 2019.09.04. 20:53 keltezéssel, Soft Works írta: This commit adds two logging flags: 'timing' and 'datetiming'. Usage: ffmpeg -logleve +timing Timing information will only be added to log lines once transcoding has started. Why this restriction? I think all log lines should be start with

[FFmpeg-devel] [PATCH] Add option to log timing

2019-09-04 Thread Soft Works
This commit adds two logging flags: 'timing' and 'datetiming'. Usage: ffmpeg -logleve +timing Timing information will only be added to log lines once transcoding has started. --- fftools/cmdutils.c | 14 ++ fftools/ffmpeg.c | 3 +++ libavutil/log.c| 37 +++