[MPlayer-dev-eng] [PATCH] changing audio_delay with OSD menu

[Davide Capodaglio]

Now it is more compact.
Sorry but I don't like the mixing of spaces and tabs used... :-)

--- command.c.orig      2008-01-04 20:43:24.000000000 +0100
+++ command.c   2008-01-04 20:44:12.000000000 +0100
@@ -507,11 +507,10 @@
        return M_PROPERTY_UNAVAILABLE;
     switch (action) {
     case M_PROPERTY_SET:
-    case M_PROPERTY_STEP_UP:
-    case M_PROPERTY_STEP_DOWN:
        if (!arg)
            return M_PROPERTY_ERROR;
-       else {
+    case M_PROPERTY_STEP_UP:
+    case M_PROPERTY_STEP_DOWN: {
            float delay = audio_delay;
            m_property_delay(prop, action, arg, &audio_delay);
            if (mpctx->sh_audio)


Guillaume LECERF ha scritto:
> 2008/1/4, Davide Capodaglio <davidecapod at inwind.it>:
>   
>> Hi all,
>> this patch fixes the audio_delay property setting in OSD menu. Currently
>> this does not work, it is not possible to change the value with STEPUP /
>> STEPDOWN.
>> The reason behind this is that arg is NULL for step up/down and so it
>> should not be checked.
>>     
>
> Your patch contains some cosmetics, please remove them.
>
>