[MPlayer-dev-eng] [PATCH] Add support for directory-wide configuration overrides V2.0

Christian P. Schmidt schmidt at digadd.de
Mon Aug 31 10:52:02 CEST 2009


Reimar Döffinger wrote:
> On Mon, Aug 31, 2009 at 10:32:19AM +0300, Christian P. Schmidt wrote:
>> Diego Biurrun wrote:
>>> On Sun, Aug 30, 2009 at 03:04:12PM +0300, Christian P. Schmidt wrote:
>>>> --- mplayer.c	(revision 29565)
>>>> +++ mplayer.c	(working copy)
>>>> @@ -916,10 +917,26 @@
>>>>  
>>>> +    if (use_filedir_conf)
>>>> +    {
>>> if (use_filedir_conf) {
>> That breaks the style currently in use in the function. Is it ok to mix
>> styles?
> 
> Sorry you already got the cosmetic comments, the more significant is
> that there is already code to find the start of the file name a few
> lines below, you should move that up and reuse instead of duplicating
> it (so I'll only have to fix one place once I find the time to test on
> Windows).

Right, redid the whole function.

Structure had to change a bit because the two strrchr() calls were made
on different variables, but it could be unified.

Regards,
Christian
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: dirwide-mplayer-conf.v2.0.patch
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20090831/d6a61029/attachment.txt>


More information about the MPlayer-dev-eng mailing list