<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Dec 21, 2012 at 4:24 PM, Igor Stasenko <span dir="ltr">&lt;<a href="mailto:siguctua@gmail.com" target="_blank">siguctua@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"><div class="im"><br>
On 21 December 2012 22:09, Eliot Miranda &lt;<a href="mailto:eliot.miranda@gmail.com">eliot.miranda@gmail.com</a>&gt; wrote:<br>
&gt;<br>
&gt; Igor,<br>
&gt;<br>
&gt;     quite so.  Perhaps I&#39;ll have time to change this soon.  SHouldn&#39;t be hard.  The test is straight-forward.  if after the change the system spits out unchanged source it is working :)<br>
&gt;<br>
</div>I can do it. Because i have a lot of pressure from Pharo team to fix VM...<br>
i don&#39;t want to rollback last commit which broken our builds but<br>
instead fix that stuff.<br>
<br>
I will create a separate .cs so that you can easily merge it with your branch.<br></blockquote><div><br></div><div>thanks!!  Happy Holidays...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im HOEnZb"><br>
&gt;<br>
&gt; On Wed, Dec 19, 2012 at 7:12 PM, Igor Stasenko &lt;<a href="mailto:siguctua@gmail.com">siguctua@gmail.com</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; yeah.. just looked again at #buildCodeGeneratorForCogit: getAPIMethods<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;         cogitClasses := OrderedCollection new.<br>
&gt;&gt;         [cogitClasses addFirst: cogitClass.<br>
&gt;&gt;          cogitClass ~~ Cogit<br>
&gt;&gt;          and: [cogitClass inheritsFrom: Cogit]] whileTrue:<br>
&gt;&gt;                 [cogitClass := cogitClass superclass].<br>
&gt;&gt;         cogitClasses addFirst: VMClass.<br>
&gt;&gt;         cogitClasses addAllLast: self cogitClass ancilliaryClasses.<br>
&gt;&gt;         cogitClasses do: [:cgc| cg addClass: cgc].<br>
&gt;&gt;         (cg structClassesForTranslationClasses: cogitClasses) do:<br>
&gt;&gt;                 [:structClass| cg addStructClass: structClass].<br>
&gt;&gt;<br>
&gt;&gt; see what wrong there?<br>
&gt;&gt;<br>
&gt;&gt; it letting vmmaker to decide for a class what to do, instead letting<br>
&gt;&gt; class to decide for itself.<br>
&gt;&gt; This logic is clearly do not belongs to right place.<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; --<br>
&gt; best,<br>
&gt; Eliot<br>
&gt;<br>
<br>
<br>
<br>
</div><div class="HOEnZb"><div class="h5">--<br>
Best regards,<br>
Igor Stasenko.<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>best,<div>Eliot</div>
</div></div>