[Clam-devel] [LADSPA] not using LWrapperConfig and using real classes on LFactoryTest

Andreas Calvo flipy.bcn at gmail.com
Thu May 24 04:20:55 PDT 2007


On 5/24/07, Pau Arumi <parumi at iua.upf.edu> wrote:
>
>
> Commited on revision:  10147
> Nice patch, it reduces a lot of complexity!
>
> However see my changes (over your patch) in LadspaFactoryTest:
> - avoid assert_equal of two char*
> - remved unnecessary std::string at parameters
> - use only noise.so and sine.so both libs comes with the
> ladspa-sdk package


thanks! I was working with my local sandbox and didn't notice which were
comming with ladspa-sdk!

> Since most of the param ara used only to load the plugin, and with the
> > changes that were made last week to the processing configuration, the
> > dependency of a LWrapperConfig has been dropped. One variable has been
> > created, mFactoryKey, which stores the name of the plugin.
> > Now Size and SampleRate are static, but it'd change in the near future.
>
> By static you mean "hardcoded". Yes this will have to change.
> Related to that I'll send a proposal on how processings could see
> the host (networkplayer) buffer size if are interested.


but, if someone needs them, then I think it'd be useful to have a
configuration for that, right?

pau


andreas

_______________________________________________
> Clam-devel mailing list
> Clam-devel at llistes.projectes.lafarga.org
> https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.clam-project.org/pipermail/clam-devel-clam-project.org/attachments/20070524/42ff314a/attachment-0003.htm>


More information about the clam-devel mailing list