[FFmpeg-devel] [PATCH] tools: add make_chlayout_test.plx.
nicolas.george at normalesup.org
Mon Jul 30 13:22:01 CEST 2012
Le duodi 12 thermidor, an CCXX, Stefano Sabatini a écrit :
> Why "plx" rather than "pl"?
.pl is the extension for the old "perl library" files; using it for scripts
was a misnomer. I have read (but can not find the source again) that Larry
Wall recommends .plx.
> Also, maybe drop the suffix, the other
> scripts in tools don't have it.
> options? Or "is a list of any valid ffmpeg option".
> This requirement is really strong, and could be avoided by exposing a
> list of channel layouts through an option/interface, and could also
> simplify much the code below (and make it more robust).
The rest is no longer relevant since the parsing is completely different.
Thanks for the review.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel