<div>Passing a class by copy may bring performance issues on realtime i think, that's why i did it by reference.</div>
<div>If you want it to be by copy, i have no problem to do it by copy.<br>For the next patch i'll change it and add the IsConnected test and pass it.</div>
<div>that's all for now</div>
<div><br> </div>
<div><span class="gmail_quote">On 5/9/08, <b class="gmail_sendername">Pau Arumí</b> <<a href="mailto:parumi@iua.upf.edu">parumi@iua.upf.edu</a>> wrote:</span>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">On dj, 2008-05-08 at 14:43 -0300, Francisco Tufró wrote:<br>> After some hours of work i have done 2 tests and passed them.<br>
> Just check them and tell me if they're ok (and if the patch is well<br>> done ^^)<br>> :)<br><br>Good start! I see the code checked in by Hernan.<br>It's is good to go in small steps. Even this simple test brings out some<br>
issues:<br><br><br>+               {<br>+                       CLAM::TypedInControl<int> in("IntInControl");<br>+                       CLAM::TypedOutControl<int> out("IntOutControl");<br>
+                       out.AddLink(in);<br>+                       int number=1;<br>+                       out.SendControl(number);<br>+                       CPPUNIT_ASSERT_EQUAL( 1 , in.GetLastValue() );<br>+               }<br>
+<br><br>out.SendControl(1); won't work because the signature wants a reference<br>to a non-constant var! And we need it to work, for backwards<br>compatibility among other reasons.<br><br>So change the test passing a constant, and pass the test by changing<br>
SendControl and DoControl arguments to be const references.<br><br>We want controls to be passed by copy (or maybe in the future:<br>copy-on-write smart pointers), but keeping a reference to the original<br>object is bad.<br>
<br>I was considering a new "sent by copy" test (e.g. send a var, change the<br>var, check that the received value has not changed) but i realize it is<br>not necessary, once we have changed the previous test: if you save a a<br>
const argument it necessarily implies doing copy.<br><br>> When you give me the ok, i'll continue with the<br>> "TypedInControl::IsConnected(out)" test.<br>> Bye!<br><br>Yes<br><br>Pau<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">https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel</a><br>
</blockquote></div><br>