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

[Reimar Döffinger]

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.
Maybe starting with support for the non-cost variants is a better idea
(esp. since those are the only ones I could test anyway).

Greetings,
Reimar Döffinger