<br><br><div class="gmail_quote">On Mon, Mar 22, 2010 at 2:54 PM, Nicolas Cellier <span dir="ltr">&lt;<a href="mailto:nicolas.cellier.aka.nice@gmail.com">nicolas.cellier.aka.nice@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
You might also want to check last code published in trunk.<br></blockquote><div><br></div><div>I saw that.  You&#39;re eliminating closeTo: for float comparison which seems fine to me :)  Thanks Nicolas!</div><div>However, it is indicative that a recompile all is necessary after redefining the number scanning facilities.  I guess that this is part of the standard release build process.  If not, we should add it.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
Nicolas<br>
<br>
2010/3/22 Eliot Miranda &lt;<a href="mailto:eliot.miranda@gmail.com">eliot.miranda@gmail.com</a>&gt;:<br>
<div><div></div><div class="h5">&gt;<br>
&gt;<br>
&gt; On Mon, Mar 22, 2010 at 2:25 PM, Stéphane Ducasse<br>
&gt; &lt;<a href="mailto:stephane.ducasse@inria.fr">stephane.ducasse@inria.fr</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; eliot<br>
&gt;&gt;<br>
&gt;&gt; Your changes fixed some of the problems. Now I tried to fix the test<br>
&gt;&gt; testAnalogousCodeTo in MethodPropertiesTest and<br>
&gt;&gt; the expression returns false.<br>
&gt;&gt;<br>
&gt;&gt; (#zork-&gt;&#39;hello&#39;) analogousCodeTo: (#zork-&gt;&#39;hello&#39;)  -&gt; false<br>
&gt;&gt;<br>
&gt;&gt; (#zork-&gt;&#39;hello&#39;) = (#zork-&gt;&#39;hello&#39;) -&gt; true<br>
&gt;<br>
&gt; For me<br>
&gt;     #(#zork -&gt; &#39;hello&#39;) size 3<br>
&gt; In Pharo is it true that<br>
&gt; #(#zork -&gt; &#39;hello&#39;) size 1<br>
&gt; ??!?!<br>
&gt; In that case I would implement analogousCodeTo: in Association, e.g.<br>
&gt; analogousCodeTo: anObject<br>
&gt;     ^anObject class == self class<br>
&gt;     and: [key = anObject key<br>
&gt;     and: [value = anObject value]]<br>
&gt;<br>
&gt; But if Association isn&#39;t a literal I would simply check<br>
&gt; MethodPropertiesTests.  It could be obsolete.<br>
&gt;&gt;<br>
&gt;&gt; Stef<br>
&gt;&gt;<br>
&gt;&gt; On Mar 22, 2010, at 6:20 PM, Eliot Miranda wrote:<br>
&gt;&gt;<br>
&gt;&gt; &gt; Oops.  Find attached.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; 2010/3/22 Cyrille Delaunay &lt;<a href="mailto:cy.delaunay@gmail.com">cy.delaunay@gmail.com</a>&gt;<br>
&gt;&gt; &gt; try in a Pharo-1.0-10515-rc3 image :<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; |set|<br>
&gt;&gt; &gt; set := Set new.<br>
&gt;&gt; &gt; Collection withAllSubclasses do: [:aClass |<br>
&gt;&gt; &gt;       set addAll: aClass methods<br>
&gt;&gt; &gt;       ].<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; It will raise an exception telling: &quot;MessageNotUnderstood:<br>
&gt;&gt; &gt; ByteSymbol&gt;&gt;analogousCodeTo:&quot;.<br>
&gt;&gt; &gt; Indeed, in the code of CompiledMethod &gt;&gt; = , the message<br>
&gt;&gt; &gt; &#39;analogousCodeTo:&#39; is send to a<br>
&gt;&gt; &gt; symbol.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; I opened an Issue:<br>
&gt;&gt; &gt; <a href="http://code.google.com/p/pharo/issues/detail?id=2185" target="_blank">http://code.google.com/p/pharo/issues/detail?id=2185</a><br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; _______________________________________________<br>
&gt;&gt; &gt; Pharo-project mailing list<br>
&gt;&gt; &gt; <a href="mailto:Pharo-project@lists.gforge.inria.fr">Pharo-project@lists.gforge.inria.fr</a><br>
&gt;&gt; &gt; <a href="http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project" target="_blank">http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project</a><br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; &lt;Object-analogousCodeTo.st&gt;_______________________________________________<br>
&gt;&gt; &gt; Pharo-project mailing list<br>
&gt;&gt; &gt; <a href="mailto:Pharo-project@lists.gforge.inria.fr">Pharo-project@lists.gforge.inria.fr</a><br>
&gt;&gt; &gt; <a href="http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project" target="_blank">http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project</a><br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; _______________________________________________<br>
&gt;&gt; Pharo-project mailing list<br>
&gt;&gt; <a href="mailto:Pharo-project@lists.gforge.inria.fr">Pharo-project@lists.gforge.inria.fr</a><br>
&gt;&gt; <a href="http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project" target="_blank">http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project</a><br>
&gt;<br>
&gt;<br>
&gt; _______________________________________________<br>
&gt; Pharo-project mailing list<br>
&gt; <a href="mailto:Pharo-project@lists.gforge.inria.fr">Pharo-project@lists.gforge.inria.fr</a><br>
&gt; <a href="http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project" target="_blank">http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project</a><br>
&gt;<br>
<br>
_______________________________________________<br>
Pharo-project mailing list<br>
<a href="mailto:Pharo-project@lists.gforge.inria.fr">Pharo-project@lists.gforge.inria.fr</a><br>
<a href="http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project" target="_blank">http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project</a><br>
</div></div></blockquote></div><br>