<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Yes. Just commit to the repo as usual and we'll build another release candidate.</div><div><br></div><div>(Karl, please don't "patch the image", let's do it properly)</div><div><br></div><span class="Apple-style-span" style="font-family: Helvetica; font-size: 12px; ">- Bert -</span><div><font class="Apple-style-span" face="Helvetica"><br></font><div><div>On 28.03.2012, at 18:12, karl ramberg wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Don't be sorry.<div>I'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">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; position: static; z-index: auto; ">I'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:&nbsp;<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&nbsp;<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,&nbsp;I'm terribly sorry.&nbsp;Now&nbsp;I don'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'm very very sorry for this inconvenience.</div><div><br></div><div><div>Richo</div><br><div class="gmail_quote"><div class="im">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 class="h5"><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; position: static; z-index: auto; "><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 'at' <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 apple-content-edited="true"><span class="Apple-style-span" style="border-collapse: separate; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; color: rgb(0, 0, 0); font-family: 'Lucida Grande'; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; -webkit-text-decorations-in-effect: none; text-indent: 0px; -webkit-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><br class="Apple-interchange-newline"></span>
</div>
<br></div></body></html>