[Clam-devel] TYPEDCONTROLS REFACTORING: Opinions about actual controls compatibility

Francisco Tufró nictuku at gmail.com
Sun Aug 10 20:59:50 PDT 2008


Maybe the first patch was bad. Test this also.

On Mon, Aug 11, 2008 at 12:50 AM, Francisco Tufró <nictuku at gmail.com> wrote:

> These are the patchs i did for the migraton.
> They're splitted by application.
> Annotator, Voice2MIDI and SMSTools needed changes that could be done
> without the final migration, (basically GetInControls issue instances) so i
> commited these.
> Here i put some info we have to discuss:
> CLAM:
>  - Problem with publishers. I think they're related with upcasting. (The
> SendControl is accessing the DoControl from TypedInControl<float> instead of
> the DoControl from InControlPublisher, and so..
> These tests were commented in order to pass the tests:
>     · testInControlPublisher
>     · testOutControlPublisher
>     · testInControlPublisher_ConnectControlsFromPublisher
>
> Turnaround:
> The problem of GetInControl in CLAM::Processing is repeated in the
> GetControlByName from CLAM::Network, should we add another function to
> handle this (like SendFloatToControl) or just review the code (maybe an
> oppinion from Pawel is needed here)? (src/Turnaround.cxx:244)
> I didn't do a patch of this since i don't know exactly what to do with it.
>
> This is all by now.
> Need feedback :)
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.clam-project.org/pipermail/clam-devel-clam-project.org/attachments/20080811/88660be1/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: TypedControlsRefactoring_CLAM_v2.patch
Type: text/x-patch
Size: 48200 bytes
Desc: not available
URL: <http://lists.clam-project.org/pipermail/clam-devel-clam-project.org/attachments/20080811/88660be1/attachment-0004.bin>


More information about the clam-devel mailing list