[Clam-devel] patch: Processing mods.

Hernán Ordiales h at ordia.com.ar
Sat Jun 7 07:59:30 PDT 2008


On Sat, Jun 7, 2008 at 7:41 AM, David García Garzón <dgarcia at iua.upf.edu> wrote:
> I am sorry, guys. I am not following your thread that close and this could
> seem random babble or just something you have already thought about.
>
> Pau told me that you were dupping the control registry so that both controls
> and typed controls will be in processings for a while. But if we keep the
> same interface, at some point you could define a
>        typedef TypedInControl<TControlData> InControl;
> so there is no need to keep a double registry for current and typed controls.
> If some of the processings are using some of the dropped features of old
> Controls then that should be a good way to spot the guilty ones.

Yes, good tip, I think we will end up doing that at some point, but
still we're not ready for that since we need to complete and test
other features first (i.e. implement the missing functions in network
class)

cheers

> Sorry if it is just random babble.
>
> David.
>
>
> On Dijous 05 Juny 2008, Francisco Tufró wrote:
>> That's right, sorry :(
>> I'm resending the patch.
>>
>> On Wed, Jun 4, 2008 at 11:16 PM, Hernán Ordiales <h at ordia.com.ar> wrote:
>> > seems promising :-D, but i think you forgot to add the .cxx file you
>> > had to add since the linking problem we chatted this afternoon...
>> > maybe TypedOutControl.cxx  (and maybe others more)? am i right?
>> >
>> > On Wed, Jun 4, 2008 at 6:01 PM, Francisco Tufró <nictuku at gmail.com> wrote:
>> > > 04/06/08: typed_controls_v10_Processing_mods.patch
>> > >     - Added the member mProcessing to Typed*Control classes and its
>> >
>> > getters.
>> >
>> > >     - Modified Typed*Control constructors to set mProcessing member.
>> > >     - Added RegisterTyped*Control, GetTyped*Control and
>> > > HasTyped*Control functions to Processing class.
>> > >     - Modified BaseTyped*Control constructors to register controls when
>> > > created.
>> > >     - Moved some code from Typed*Control.hxx to Typed*Control.cxx to
>> >
>> > solve
>> >
>> > > duplicate link problem.
>> > >     - Added GetByNumber, Size() and Has functions to
>> >
>> > Typed*ControlRegistry
>> >
>> > > classes.
>> > >     - Added mTyped*ControlRegistry members and Get*Controls functions
>> > > to Processing class
>> > >     - Wrote and passed:
>> > >         - testGetTypedInControl_GetTheRightControl
>> > >         - testGetTypedOutControl_GetTheRightControl
>> > >         - testGetTypedInControl_WithOutOfRangeIndexThrowException
>> > >         - testGetTypedOutControl_WithOutOfRangeIndexThrowException
>> > >         - testLinkAndSendControl_ChangesTypedInControlState
>> > >         - testTypedInControls_GetByNumber_GetTheRightControl
>> > >         - testTypedOutControls_GetByNumber_GetTheRightControl
>> > >         - testTypedInControls_GetByName_GetTheRightControl
>> > >         - testTypedOutControls_GetByName_GetTheRightControl
>> > >         - testTypedInControls_Size
>> > >         - testTypedOutControls_Size
>> > >         - testConnectTypedControls_usingNames
>> > >         - testConnectTypedControls_usingNumbers
>> > >
>> > >
>> > >
>> > > _______________________________________________
>> > > Clam-devel mailing list
>> > > Clam-devel at llistes.projectes.lafarga.org
>> >
>> > https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel
>> >
>> >
>> >
>> >
>> >
>> > --
>> > Hernán
>> > http://h.ordia.com.ar
>> > GnuPG: 0xEE8A3FE9
>> >
>> > _______________________________________________
>> > Clam-devel mailing list
>> > Clam-devel at llistes.projectes.lafarga.org
>> > https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel
>
>
>
> --
> David García Garzón
> (Work) dgarcia at iua dot upf anotherdot es
> http://www.iua.upf.edu/~dgarcia
>
> _______________________________________________
> Clam-devel mailing list
> Clam-devel at llistes.projectes.lafarga.org
> https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel
>
>



-- 
Hernán
http://h.ordia.com.ar
GnuPG: 0xEE8A3FE9




More information about the clam-devel mailing list