Ok<div>Update is published in the update stream.</div><div>Your video on how to do a update is invaluable :-)</div><div><br></div><div><br></div><div>Karl</div><div><br><br><div class="gmail_quote">On Wed, Mar 28, 2012 at 7:34 PM, karl ramberg <span dir="ltr">&lt;<a href="mailto:karlramberg@gmail.com">karlramberg@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">Yes<div>I&#39;ll start now</div><div><br></div><div>Karl</div><div><br><br><div class="gmail_quote"><div><div class="h5">
On Wed, Mar 28, 2012 at 6:31 PM, Bert Freudenberg <span dir="ltr">&lt;<a href="mailto:bert@freudenbergs.de" target="_blank">bert@freudenbergs.de</a>&gt;</span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div style="word-wrap:break-word"><div>Yes. Just commit to the repo as usual and we&#39;ll build another release candidate.</div>

<div><br></div><div>(Karl, please don&#39;t &quot;patch the image&quot;, let&#39;s do it properly)</div><span><font color="#888888"><div><br></div><span style="font-family:Helvetica;font-size:12px">- Bert -</span></font></span><div>

<div><div><font face="Helvetica"><br></font><div><div>On 28.03.2012, at 18:12, karl ramberg wrote:</div><br><blockquote type="cite">Don&#39;t be sorry.<div>I&#39;ll add the changes to the image</div><div><br></div>
<div>Karl<br><br><div class="gmail_quote">On Wed, Mar 28, 2012 at 4:30 PM, Ricardo Moran <span dir="ltr">&lt;<a href="mailto:richi.moran@gmail.com" target="_blank">richi.moran@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">I&#39;m sorry to say that while testing something else I found two bugs with the step button.<div>

<br></div><div>The first one fails when trying to evaluate a test condition involving a function (like abs, random, and such). The fix is trivial, but I might be missing something:<div>


<br></div><div><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">




CompoundTileMorph&gt;&gt;evaluateTestPart<br><span style="white-space:pre-wrap">        </span>| condition |<br><span style="white-space:pre-wrap">        </span>condition := testPart tiles at: 1 ifAbsent: [^ true].<br>

<span style="white-space:pre-wrap">        </span>self halt.<br><span style="white-space:pre-wrap">        </span>^ Compiler evaluate: condition codeString<br><span style="white-space:pre-wrap">                                </span>for: <b>(condition associatedPlayer<br>




</b><b><span style="white-space:pre-wrap">                                        </span>ifNil: [condition topEditor playerScripted])<br></b><span style="white-space:pre-wrap">                                </span>logged: false</blockquote><div><br>

</div>The second one is worst because it hangs the image. To reproduce just try to step on a script with an empty test tile. The fix is simple as well:</div><div><br></div><div><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">




EtoysDebugger&gt;&gt;evaluateTest: test<br><span style="white-space:pre-wrap">        </span>| tile |<br><span style="white-space:pre-wrap">        </span>test testPart tiles isEmpty<br><span style="white-space:pre-wrap">                </span>ifTrue: [next := test yesPart tiles at: 1 ifAbsent: [test nextTile].<br>




<span style="white-space:pre-wrap">                        </span><b>next = test <br><span style="white-space:pre-wrap">                                </span>ifTrue: [^ self]<br><span style="white-space:pre-wrap">                                </span>ifFalse: [^ self evaluateNextTile]</b>].<br>




<span style="white-space:pre-wrap">        </span>self highlight: test testPart.<br><span style="white-space:pre-wrap">        </span>tile := test evaluateTestPart<br><span style="white-space:pre-wrap">                </span>ifTrue: [test yesPart]<br>




<span style="white-space:pre-wrap">                </span>ifFalse: [test noPart].<br><span style="white-space:pre-wrap">        </span>next := tile tiles at: 1 ifAbsent: [test nextTile]</blockquote><div><br>

</div><div>I should have found these bugs earlier, I&#39;m terribly sorry. Now I don&#39;t know how to proceed. I will open a ticket of course, but should I attach a change set there or can I commit the fix to etoys? I think this needs to be fixed in 5.0.</div>




<div><br></div><div>Again, I&#39;m very very sorry for this inconvenience.</div><div><br></div><div><div>Richo</div><br><div class="gmail_quote"><div>On Wed, Mar 28, 2012 at 8:51 AM, Bert Freudenberg <span dir="ltr">&lt;<a href="mailto:bert@freudenbergs.de" target="_blank">bert@freudenbergs.de</a>&gt;</span> wrote:<br>




</div><div><div><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div>On 27.03.2012, at 08:59, karl ramberg wrote:<br>
<br>
&gt; Etoys 5 release candidate 2<br>
&gt; More fixes went into this new release candidate, but if there are some show stoppers we would like to know asap !<br>
<br>
</div>Actually it would be good to hear if anyone tested this, and it worked.<br>
<br>
- Bert -<br>
<div><br>
<br>
&gt; So download from here:<br>
&gt;<br>
&gt; <a href="http://squeakland.org/download/" target="_blank">http://squeakland.org/download/</a><br>
&gt;<br>
&gt; And give feedback<br>
&gt;<br>
&gt; <a href="http://tracker.squeakland.org/" target="_blank">http://tracker.squeakland.org</a><br>
&gt;<br>
&gt; or to<br>
&gt;<br>
&gt; <a href="mailto:etoys-dev@squeakland.org" target="_blank">etoys-dev@squeakland.org</a><br>
&gt;<br>
&gt; last resort is mailing me directly karlramberg &#39;at&#39; <a href="http://gmail.com/" target="_blank">gmail.com</a><br>
&gt;<br>
&gt; Cheers,<br>
&gt; Karl</div></blockquote></div></div></div></div></div></div></blockquote></div><br></div>
</blockquote></div><div><span style="line-height:normal;text-indent:0px;border-collapse:separate;letter-spacing:normal;font-variant:normal;text-transform:none;font-style:normal;white-space:normal;font-family:&#39;Lucida Grande&#39;;font-weight:normal;word-spacing:0px"><br>

</span>
</div>
<br></div></div></div></div><br></div></div>_______________________________________________<br>
etoys-dev mailing list<br>
<a href="mailto:etoys-dev@squeakland.org" target="_blank">etoys-dev@squeakland.org</a><br>
<a href="http://lists.squeakland.org/mailman/listinfo/etoys-dev" target="_blank">http://lists.squeakland.org/mailman/listinfo/etoys-dev</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>