[vlc-commits] aout: inline one function and simplify
    Rémi Denis-Courmont 
    git at videolan.org
       
    Sun Apr  7 17:45:51 CEST 2013
    
    
  
vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Sun Apr  7 18:45:12 2013 +0300| [edada22bdb0c45d13d5c37450d25d60b3d4867e8] | committer: Rémi Denis-Courmont
aout: inline one function and simplify
> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=edada22bdb0c45d13d5c37450d25d60b3d4867e8
---
 src/audio_output/filters.c |   25 +++++++------------------
 1 file changed, 7 insertions(+), 18 deletions(-)
diff --git a/src/audio_output/filters.c b/src/audio_output/filters.c
index 5e48f6a..b29833b 100644
--- a/src/audio_output/filters.c
+++ b/src/audio_output/filters.c
@@ -250,12 +250,6 @@ error:
 #define aout_FiltersPipelineCreate(obj,f,n,m,i,o) \
         aout_FiltersPipelineCreate(VLC_OBJECT(obj),f,n,m,i,o)
 
-static inline bool ChangeFiltersString (vlc_object_t *aout, const char *var,
-                                        const char *filter, bool add)
-{
-    return aout_ChangeFilterString (aout, aout, var, filter, add);
-}
-
 /**
  * Filters an audio buffer through a chain of filters.
  */
@@ -300,26 +294,21 @@ static int VisualizationCallback (vlc_object_t *obj, const char *var,
     return VLC_SUCCESS;
 }
 
-static int EqualizerCallback (vlc_object_t *obj, char const *var,
+static int EqualizerCallback (vlc_object_t *obj, const char *var,
                               vlc_value_t oldval, vlc_value_t newval,
                               void *data)
 {
-    audio_output_t *aout = (audio_output_t *)obj;
-    char *mode = newval.psz_string;
-    bool ret;
+    const char *val = newval.psz_string;
 
-    if (!*mode)
-        ret = ChangeFiltersString (obj, "audio-filter", "equalizer", false);
-    else
+    if (*val)
     {
         var_Create (obj, "equalizer-preset", VLC_VAR_STRING);
-        var_SetString (obj, "equalizer-preset", mode);
-        ret = ChangeFiltersString (obj, "audio-filter", "equalizer", true);
+        var_SetString (obj, "equalizer-preset", val);
     }
 
-    /* That sucks */
-    if (ret)
-        aout_InputRequestRestart (aout);
+    if (aout_ChangeFilterString (obj, obj, "audio-filter", "equalizer", *val))
+        aout_InputRequestRestart ((audio_output_t *)obj); /* <- That sucks! */
+
     (void) var; (void) oldval; (void) data;
     return VLC_SUCCESS;
 }
    
    
More information about the vlc-commits
mailing list