[MPlayer-dev-eng] [PATCH] -noconfig option
Andrew Savchenko
Bircoph at list.ru
Tue Apr 15 21:40:56 CEST 2008
Hi,
On Monday 14 April 2008 15:54, Alban Bedel wrote:
[...]
> > The first patch is a bugfix for mencoder: system-wide config
> > file reading.
>
> Applied with a fix, declaration are not allowed in the middle of
> code blocks (it break gcc 2.95.3).
Yes, I must remember at some point that mplayer must be compatible
with gcc-2.95...
> > The second patch is -noconfig option itself, requires 1st
> > path.
>
> Applied with a few fixes: preparsed option _must_ be global
I agree with this.
> and the noconfig_all function can be made static.
Well, I do not object against this. But what is the reason to make
it static? It has no arguments neither variable declarations, thus
I can see no benefits nor disadvantages of making it static.
Sincerely,
Andrew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080415/d996c0c3/attachment.pgp>
More information about the MPlayer-dev-eng
mailing list