[FFmpeg-devel] [PATCH] avutil: expand the size of LINE_SZ to 2048.

Nicolas George nicolas.george at normalesup.org
Sat Aug 10 10:22:21 CEST 2013


Le tridi 23 thermidor, an CCXXI, Stephen Hutchinson a écrit :
> Unfortunately, my ideas about how to get it to print one option per line
> didn't really pan out since I was just messing around with sed*, and

Why don't you just do it in C? For example, use strcspn/strchr to find the
length of the next word, and then use this to print it:

	av_log(..., "%.*s ", length, str);

> there's no way of predicting that sed's behavior will remain the same
> across the different versions of it used on various platforms

sed is standardized; if you carefully adhere to the standard, you can do
pretty advanced stuff. Not that I think this is a good idea, it would be
unreadable.

Also, I do not believe that printing one option per line is a good idea; my
usual build have 36 build options, mostly --enable-foo: that is exactly the
size of most of my work ttys.

I can suggest this: replace "--enable-foo" by "+foo", and wrap to 76
columns.

Regards,

-- 
  Nicolas George
-------------- 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/20130810/42b76f9f/attachment.asc>


More information about the ffmpeg-devel mailing list