[FFmpeg-devel] [PATCH] avutil: expand the size of LINE_SZ to 2048.
eclipse7 at gmx.net
Thu Aug 8 02:02:10 CEST 2013
On 2013-08-07 05:11 -0400, Stephen Hutchinson wrote:
> On Wed, Aug 7, 2013 at 4:49 AM, Michael Niedermayer <michaelni at gmx.at>wrote:
> > maybe the way FFMPEG_CONFIGURATION is printed could be changed so
> > its nicer formated and then also is printed in smaller parts not
> > needing this.
> > I dont think 2048 is enough, one could arbitrarily repeat --enables
> > or list all codecs explicitly or have very long pathes, ...
> > and always increasing LINE_SZ would eventually cause problems with
> > embeded systems that have only little stack space (such do exist
> > AFAIK)
> I was thinking that would be the problem. My other inclination would be to
> remove the configuration report from a plain invocation, but add an option
> under help that would display it, optionally in a better-formatted way
> (like one option per line or grouped into categories like the options
> appear in configure). But I wasn't quite sure where to look in the code to
> find how to do that.
It might be rather unfortunate to remove the configure line from
normal invocation. It speeds up trouble shooting and can be helpful
when working on bug reports.
Printing it in a nicer way and splitting over multiple lines sounds
like a good idea though. But try to do it in the simplest and robust
way you can think of. Over-engineering won't add much value here and
some trouble in the worst case.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: not available
More information about the ffmpeg-devel