[Clam-devel] Bug solved for deleting Out In Ports and Out In Controls

Ferran Orriols ferran.orriols at gmail.com
Thu Jul 17 01:03:49 PDT 2008


Sorry Pau, next patch I will try being more specific. Ok, I will write some
documentation for the Ports/Controls classes and an example how to work with
dynamic ports in the wiki. I have added this in my TODO's list.



2008/7/16 Pau Arumí <parumi at iua.upf.edu>:

> On dc, 2008-07-16 at 15:55 +0200, Ferran Orriols wrote:
> > Hi Clam,
> >
> > I found a bug when you want to delete any port or control. I have
> > solved the bug and I have written the tests for each port and control
> > and it pass. I have attached the patch with the new tests in
> > CLAM/test/UnitTests/ProcessingBaseTests/ProcessingTest.cxx and the
> > changes in the next files for solved the problem.
>
> Here goes the commit. Ferran try being more specific on what the patch
> does next time.
>
> Commit 11614
>
>  * Ports/Controls fix on destructor: now, on deletion gets unregistered
> from the Processing base registry. This enables increasing/decreasing
> the number of ports maintaining connections. (Before you had to do
> GetInPorts().Clear(), loosing all connections).
>   Pair-programmed with Ferran
>
>
> BTW, adding a line or two of documentation in the Ports/Controls classes
> would not hurt. I mean, specifying the semantics of the constructor and
> destructor regarding the registration with the Processing. Also, keep us
> informed if you decide to write a "dynamic ports" example in the wiki.
> Thanks :-)
>
> P
>
>
>
>
> _______________________________________________
> Clam-devel mailing list
> Clam-devel at llistes.projectes.lafarga.org
> https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel
>



-- 
Ferran Orriols
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.clam-project.org/pipermail/clam-devel-clam-project.org/attachments/20080717/946e24d4/attachment.html>


More information about the clam-devel mailing list