[FFmpeg-devel] Per codec options? (was: [PATCH] libmp3lame: set noise shaping & psycho acoustic algorithms quality)

[Michael Niedermayer]

On Thu, Jul 31, 2008 at 09:26:37PM +0200, Nicolas George wrote:
> Le quartidi 14 thermidor, an CCXVI, Michael Niedermayer a ?crit?:
> > you are moving 10min of your time to 10min * 100000users thats not reasonable.
> > When each codec uses a totally different option name, its the users problem
> > to find out which to use, otherwise he obviously knows it already.
> 
> That is true for options that exist in all codecs, or most of them, like -ab
> or -aq. But there are features that exist only in one codec, and have little
> resemblance to any feature in another codec.
> 
> In the particular case of the option of this thread, for example, the people
> who want to change it are people who know LAME and know that such a setting
> exists. What they need to know is which one of the many options of ffmpeg
> will change it. Unfortunately, the name "compression_level" does not make it
> obvious at all, and no other existing option would.

a simple section in the docs that contains the word lame would make it very
easy to find for many users.


> 
> Of course, most options can, and should, be handled in a common way. But
> some can not. I think per-codecs options are the right way to deal with
> features that are really unique to a particular codec.
> 
> By the way, this part of the discussion is not really related to the patch
> itself, rather to hypothetically possible future development. I submitted a
> new version of the patch using "compression_level" as you requested: what do
> you think of it?

i think ive replied to that as well, have i not?

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I wish the Xiph folks would stop pretending they've got something they
do not.  Somehow I fear this will remain a wish. -- M?ns Rullg?rd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080801/20e26dec/attachment.pgp>