[MPlayer-dev-eng] Re: [PATCH] CoreAVC [1/5] - Add registry

[Alan Nisota]

On 10/5/06, Reimar Döffinger wrote:
> Hello,
> On Thu, Oct 05, 2006 at 09:10:17AM +0200, Tomas Janousek wrote:
> > Alan Nisota napsal(a):
> > >This is the 1st of 5 patches to support the CoreAVC.
> > >This patch implements the capability to store registry information in
> > >codecs.conf.  The patch is required to use CoreAVC, but there is
> > >nothing CoreAVC specific about it.
> >
> > It is a bit uncomfortable to store user-specific settings in codecs.conf.
>
> IMHO it is not acceptable. A "normal" user should never have a custom
> codecs.conf, it causes too much troubles esp. on updates.
They aren't 'user-specific' they are 'codec-specific'.  I.e. I would
expect that if you have the CoreAVC codec installed, you would have
these entries as well.  If you want to offer up another way to
accomplish the same thing, I'll be happy to go that way, but I was
trying not to reinvent the wheel.

> Maybe starting with support for the non-cost variants is a better idea
> (esp. since those are the only ones I could test anyway).
There is no such thing for CoreAVC