Thanks !<br><br>For the labels I didn't find the exact frequencies on the Ladspa effect website so I preferred to not be too specific... And having a single value for each controlrow seemed to work, I supposed it was applied to both channels.<br>

<br>Anyways, thanks !<br><br>Cheers,<br>Hugh<br><br><div class="gmail_quote">2010/10/20 jb <span dir="ltr"><<a href="mailto:jb@kdenlive.org">jb@kdenlive.org</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div><div></div><div class="h5">On Sunday 17 October 2010 15:34:04 Hugh Tebby wrote:<br>
> I just added support for the LADSPA 1197 effect (15 band equalizer). Here's<br>
> the xml file, and the patch making the required changes to CMakeLists and<br>
> initeffects.cpp. I also changed the "Equalizer" effect name  to "3 band<br>
> equalizer".<br>
><br>
> Could someone check this is ok and commit if it is so ? Thanks.<br>
><br>
> The code changes required for Ladspa effects are a bit cumbersome, and<br>
> there seems to be a leasr one unused fonction (ladspaEffectString()). I'll<br>
> see of I can do a quick cleanup around there and send a patch...<br>
<br>
</div></div>Thanks. I committed your patch with a few changes:<br>
<br>
in the Kdenlive xml effect description, I put labels for the 15 bands (50Hz,<br>
100Hz, ...).<br>
<br>
In initeffects.cpp, the xml string for the Ladspa effect was not correct, we<br>
need 2 values for each <controlrow> (one for each channel I guess). You can<br>
check my commit for further details (svn rev. 5026 and 5027).<br>
<br>
I agree that adding a Ladspa effect to Kdenlive is far from optimal... the code<br>
is rather old. Maybe it would be easier to have the ladspa xml string in a<br>
separate file instead of hardcoded...<br>
<br>
Anyways, if you feel like doing some cleanup, you're welcome.<br>
<br>
regards<br>
jb<br>
<br>
------------------------------------------------------------------------------<br>
Download new Adobe(R) Flash(R) Builder(TM) 4<br>
The new Adobe(R) Flex(R) 4 and Flash(R) Builder(TM) 4 (formerly<br>
Flex(R) Builder(TM)) enable the development of rich applications that run<br>
across multiple browsers and platforms. Download your free trials today!<br>
<a href="http://p.sf.net/sfu/adobe-dev2dev" target="_blank">http://p.sf.net/sfu/adobe-dev2dev</a><br>
_______________________________________________<br>
Kdenlive-devel mailing list<br>
<a href="mailto:Kdenlive-devel@lists.sourceforge.net">Kdenlive-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/kdenlive-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/kdenlive-devel</a><br>
</blockquote></div><br>