[Clam-devel] patch: Processing mods.

[David García Garzón]

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.

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.clam-project.org/pipermail/clam-devel-clam-project.org/attachments/20080607/3f6df940/attachment.sig>