[FFmpeg-devel] [PATCH 1/2] cmdutils: add -layouts option.

[Nicolas George]

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.

Will do.

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/20120827/c5dd4be7/attachment.asc>