[MPlayer-dev-eng] [PATCH] Do not read codecs.conf files by default

[Reimar Doeffinger]

Hello,
On Wed, Jan 17, 2007 at 12:16:37AM +0100, Diego Biurrun wrote:
> On Mon, Jan 15, 2007 at 11:01:34PM +0100, Reimar Döffinger wrote:
> > A possible consideration would be to change codecs.conf in the sources
> > so that it would not load without manual modification by the user,
> > a I-know-what-I-am-doing similar to the lavf bframes thing...
> 
> That's precisely what my patch does :)  It forces the user to provide a
> command line option (or config file entry) if a codecs.conf file is to
> be loaded, otherwise it is not loaded...

No, it's not precisely the same, since 1) your patch requires modifying
a _different_ file 2) as you say, it makes codecs.conf a non-config
files. That is fine. But your patch does not tell the user about this
fact! I am almost sure that next we would get bug reports about users
editing codec.conf to get some new windows codec working but it doesn't
work...

Greetings,
Reimar Döffinger