<br><br><div><span class="gmail_quote">On 5/24/07, <b class="gmail_sendername">Pau Arumi</b> <<a href="mailto:parumi@iua.upf.edu">parumi@iua.upf.edu</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>Commited on revision:  10147<br>Nice patch, it reduces a lot of complexity!<br><br>However see my changes (over your patch) in LadspaFactoryTest:<br>- avoid assert_equal of two char*<br>- remved unnecessary std::string at parameters
<br>- use only noise.so and sine.so both libs comes with the<br>ladspa-sdk package</blockquote><div><br>thanks! I was working with my local sandbox and didn't notice which were comming with ladspa-sdk!<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
> Since most of the param ara used only to load the plugin, and with the<br>> changes that were made last week to the processing configuration, the<br>> dependency of a LWrapperConfig has been dropped. One variable has been
<br>> created, mFactoryKey, which stores the name of the plugin.<br>> Now Size and SampleRate are static, but it'd change in the near future.<br><br>By static you mean "hardcoded". Yes this will have to change.
<br>Related to that I'll send a proposal on how processings could see<br>the host (networkplayer) buffer size if are interested.</blockquote><div><br>but, if someone needs them, then I think it'd be useful to have a configuration for that, right?
<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">pau</blockquote><div><br>andreas <br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
_______________________________________________<br>Clam-devel mailing list<br><a href="mailto:Clam-devel@llistes.projectes.lafarga.org">Clam-devel@llistes.projectes.lafarga.org</a><br><a href="https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel">
https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel</a><br></blockquote></div><br>