<div class="gmail_quote">On 12 August 2012 13:32, 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">
Le dimanche 12 août 2012 12:07:57 Mark Lee, vous avez écrit :<br>
<div class="im">> > > +    var_AddCallback (p_parent, "equalizer-preamp", var_Copy, aout );<br>
> > > +    var_AddCallback (p_parent, "equalizer-bands", var_Copy, aout );<br>
> > > +<br>
> ><br>
> > You need to ensure the aout variables exist, and initialized, to do that.<br>
> ><br>
> > OK, this is the only comment I'm not sure about. I have seen in<br>
><br>
> equalizer.c that it creates those variables on the aout, but of course the<br>
> equalizer may not be enabled yet. Am I OK to simply create and initialise<br>
> those variables in the same place I register the callbacks?<br>
<br>
</div>I don't know, probably. It depends on assumption made or not made within the<br>
VLC code base.<br>
<div class="im"><br>
> Or will that interfere with what equalizer.c is doing?<br>
<br>
</div>That's one of the potential issues.<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div> </div><div>Thanks, I think I have a solution for this. We'll see if it's acceptable soon.<br>
</div></div><br>