Hello...<br><br><div class="gmail_quote">On 13 July 2013 20:39, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wednesday 10 July 2013 20:44:54 Mark Lee wrote:<br>
> > > diff --git a/src/audio_output/filters.c b/src/audio_output/filters.c<br>
> > > index 41b23b4..ea451a3 100644<br>
> > > --- a/src/audio_output/filters.c<br>
> > > +++ b/src/audio_output/filters.c<br>
> > > @@ -314,8 +314,7 @@ static int EqualizerCallback (vlc_object_t *obj,<br>
> ><br>
> > const<br>
> ><br>
> > > char *var, void *data)<br>
> > ><br>
> > >  {<br>
> > ><br>
> > >      const char *val = newval.psz_string;<br>
> > ><br>
> > > -<br>
> > > -    if (*val)<br>
> > > +    if (!strcmp("equalizer", var) && *val)<br>
> ><br>
> > Tautology.<br>
><br>
> Sorry, I don't understand this.<br>
<br>
</div>If var compares equal to "equalizer", then *var equals 'e' and is thus<br>
necessarily true.<br></blockquote><div><br>But "var" and "val" are different things. I never reference "*var".<br><br>"var" is the name of the vlc variable that triggered the callback.<br>
<br>"val" is it's new value.<br><br>When var compares equal to "equalizer", then *val is either \0 or the<br>(first byte of the) name of a preset.<br><br>When var compares equal to "equalizer-bands", then *val is either \0 or<br>
the (first byte of the) concatenation of equalizer band frequency <br>values.<br><br>Not directly on point, just for context: that "equalizer" variable is<br>existing functionality, and is not related to this patch - but rather than<br>
duplicate a callback for the new patch, I re-used the existing callback<br>(conditionally).<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">
--<br>
Rémi Denis-Courmont<br>
<a href="http://www.remlab.net/" target="_blank">http://www.remlab.net/</a><br>
<br>
</div>_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</blockquote></div><br><br><a href="http://apricasoftware.co.uk" target="_blank"></a>