[FFmpeg-devel] [FFmpeg-cvslog] vf_showinfo: minimum widths for some early fields

[Clément Bœsch]

On Wed, Mar 04, 2015 at 10:18:07PM +0100, Michael Niedermayer wrote:
> On Wed, Mar 04, 2015 at 09:59:35PM +0100, Nicolas George wrote:
> > Le quartidi 14 ventôse, an CCXXIII, Peter Cordes a écrit :
> > > -           "n:%"PRId64" pts:%s pts_time:%s pos:%"PRId64" "
> > > +           "n:%4"PRId64" pts:%7s pts_time:%-7s pos:%9"PRId64" "
> > 
> > Unless I am wrong reading the printf specifiers, this will print
> > "pts:    902" instead of "pts:902". Some scripts may use the output of
> > showinfo and are not expecting the extra spaces.
> > 
> > Sorry to not have spotted that before it was applied.
> 
> hmm
> what do you suggest we do ?
> revert it ?
> is this really a problem ?
> 

Users should use ffprobe for a stable output. showinfo is more a debug
tool than anything else, its output is not stable and never was.

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150304/d9cec6f2/attachment.asc>