[FFmpeg-devel] [PATCH 1/2] cmdutils: add -layouts option.
nicolas.george at normalesup.org
Mon Aug 27 12:32:03 CEST 2012
Le primidi 11 fructidor, an CCXX, Stefano Sabatini a écrit :
> I still don't like this approach, on the ground that:
> 1. output is not easily parsable (two lists with logically different
> fields, and headers to be kept into account)
> 2. the two lists are semantically distinct
Your arguments were clearly stated, as were mine, and they were taken into
account in the remark: separated lists can be implemented (later) as another
pair of options.
I think there is a misunderstanding about what these options are meant for.
I see these options as primarily intended for users, and secondarily
parsable for scripts. See below.
> I won't oppose this patch if you insist with it, but I relaunch the
> idea of using a global "-list" option, which could work taking a list
> of things to print, e.g.:
> -list channels,layouts
> and could be extended to list the other listable things in ff*.
This is a very good idea, and I add this item: use the (future) writer API
for that, so that it can be produced in an even more script-friendly format.
> for easier parsing the header could be print to stderr (but keep
> consistency in each case).
That would mean that someone doing "ffmpeg -layouts | less" will not have
the headers. I do not think this is a good idea.
> Mention that it prints also information about the single channels.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel