[FFmpeg-devel] [PATCH 4/4] cmdutils: add -channel_layouts option.

Nicolas George nicolas.george at normalesup.org
Mon Jul 30 12:09:15 CEST 2012


Le tridi 13 thermidor, an CCXX, Stefano Sabatini a écrit :
> I mean in case you only need to show the channels *or* the standard
> layouts, having to parse a single list with a consistent structure is
> simpler than having to skip initial or ending lines based on a
> possibly changing pattern.
> 
> So it seems more natural to me to keep two distinct options for two
> different lists (that's similar to what we did with -formats ->
> -formats | -codecs | -protocols).

There are some merits to these arguments, but, on the other hand:

"ffmpeg -foos" usually lists the possible values for the "-foo" option
(codecs and formats are small exceptions: as they are very common they are
abbreviated; protocols is a bigger exception). Individual channel names as
well as standard channel layouts names serve as arguments for
-channel_layout and anywhere a channel layout is expected; indeed,
individual channel names are not accepted as argument to the "-channel" or
"-ac".

I just pushed patches 1-3, I'll wait for a third opinion on this one while I
am rebasing the channel test generation script on top of it. Thanks for the
review.

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/20120730/2a355673/attachment.asc>


More information about the ffmpeg-devel mailing list