<div dir="ltr">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.<br>
<br> <br><br><div class="gmail_quote">2008/7/16 Pau Arumí <<a href="mailto:parumi@iua.upf.edu">parumi@iua.upf.edu</a>>:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On dc, 2008-07-16 at 15:55 +0200, Ferran Orriols wrote:<br>
> Hi Clam,<br>
><br>
> I found a bug when you want to delete any port or control. I have<br>
> solved the bug and I have written the tests for each port and control<br>
> and it pass. I have attached the patch with the new tests in<br>
> CLAM/test/UnitTests/ProcessingBaseTests/ProcessingTest.cxx and the<br>
> changes in the next files for solved the problem.<br>
<br>
</div>Here goes the commit. Ferran try being more specific on what the patch<br>
does next time.<br>
<br>
Commit 11614<br>
<br>
 * Ports/Controls fix on destructor: now, on deletion gets unregistered<br>
from the Processing base registry. This enables increasing/decreasing<br>
the number of ports maintaining connections. (Before you had to do<br>
GetInPorts().Clear(), loosing all connections).<br>
   Pair-programmed with Ferran<br>
<br>
<br>
BTW, adding a line or two of documentation in the Ports/Controls classes<br>
would not hurt. I mean, specifying the semantics of the constructor and<br>
destructor regarding the registration with the Processing. Also, keep us<br>
informed if you decide to write a "dynamic ports" example in the wiki.<br>
Thanks :-)<br>
<br>
P<br>
<br>
<br>
<br>
<br>
_______________________________________________<br>
Clam-devel mailing list<br>
<a href="mailto:Clam-devel@llistes.projectes.lafarga.org">Clam-devel@llistes.projectes.lafarga.org</a><br>
<a href="https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel" target="_blank">https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>Ferran Orriols
</div>