<div dir="ltr">Ok fixed in 2282 ! Thanks for reporting</div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 26, 2017 at 11:04 PM, Clément Bera <span dir="ltr"><<a href="mailto:bera.clement@gmail.com" target="_blank">bera.clement@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">This is correct: (lessOrEqual instead of less):<div><br></div><div><span class=""><div> "0 >= start, fail"</div><div><span style="white-space:pre-wrap"> </span>cogit CmpCq: (objectMemory integerObjectOf: 0) R: startReg.</div></span><div><span style="white-space:pre-wrap"> </span>jumpOutOfBounds1 := cogit JumpLessOrEqual: 0.</div><span class=""><div><span style="white-space:pre-wrap"> </span></div><div><span style="white-space:pre-wrap"> </span>"0 >= replStart, fail"</div><div><span style="white-space:pre-wrap"> </span>cogit CmpCq: (objectMemory integerObjectOf: 0) R: repStartReg.</div></span><div><span style="white-space:pre-wrap"> </span>jumpOutOfBounds2 := cogit JumpLessOrEqual: 0.</div><div><br></div><div>I can't commit right now (my image has many changes I need to commit and I cannot branch easily on monticello...) Will do tomorrow. Thanks for reporting. </div><div><div class="h5"><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 26, 2017 at 10:58 PM, Clément Bera <span dir="ltr"><<a href="mailto:bera.clement@gmail.com" target="_blank">bera.clement@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">I think those lines are the problem:<div><br></div><div><div> "0 >= start, fail"</div><div><span style="white-space:pre-wrap"> </span>cogit CmpCq: (objectMemory integerObjectOf: 0) R: startReg.</div><div><span style="white-space:pre-wrap"> </span>jumpOutOfBounds1 := cogit JumpLess: 0.</div><div><span style="white-space:pre-wrap"> </span></div><div><span style="white-space:pre-wrap"> </span>"0 >= replStart, fail"</div><div><span style="white-space:pre-wrap"> </span>cogit CmpCq: (objectMemory integerObjectOf: 0) R: repStartReg.</div><div><span style="white-space:pre-wrap"> </span>jumpOutOfBounds2 := cogit JumpLess: 0.</div></div><div><br></div><div>If this is equal if should jump out of bounds.</div><div><br></div><div><br></div></div><div class="m_-7827408383984974839gmail-HOEnZb"><div class="m_-7827408383984974839gmail-h5"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 26, 2017 at 9:53 PM, Clément Bera <span dir="ltr"><<a href="mailto:bera.clement@gmail.com" target="_blank">bera.clement@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Bug seems to be with underflow access on replacement array:<div><br></div><div>(1 to: 10) collect: [:i| ['123456789' replaceFrom: 1 to: 4 with: 'abcdefgh' startingAt: 0] on: Error do: ['error']]<br></div><div>#('error' ' abc56789' ' abc56789' ' abc56789' ' abc56789' ' abc56789' ' abc56789' ' abc56789' ' abc56789' ' abc56789').<br></div><div><br></div><div>Same thing on arrays since this is common code...</div><div><br></div><div>Generated code looks ok... I need to look again I may have inverted some branches.</div><div><br></div><div><br></div><div><br></div><div><br></div></div><div class="m_-7827408383984974839gmail-m_-1645267237519986444HOEnZb"><div class="m_-7827408383984974839gmail-m_-1645267237519986444h5"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 26, 2017 at 9:37 PM, Clément Bera <span dir="ltr"><<a href="mailto:bera.clement@gmail.com" target="_blank">bera.clement@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Ok I have a look.<div><br></div><div>This is on byte objects...</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 26, 2017 at 9:07 PM, Nicolas Cellier <span dir="ltr"><<a href="mailto:nicolas.cellier.aka.nice@gmail.com" target="_blank">nicolas.cellier.aka.nice@gmai<wbr>l.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br><div dir="ltr"><div><div>Like reported on travis<br><a href="https://travis-ci.org/OpenSmalltalk/opensmalltalk-vm/jobs/307544570" target="_blank">https://travis-ci.org/OpenSmal<wbr>ltalk/opensmalltalk-vm/jobs/30<wbr>7544570</a><br><br></div>the testTextReplacement3 now fails randomly.<br></div>It sometimes omit to signal an Error...<br></div>
<br></blockquote></div><span class="m_-7827408383984974839gmail-m_-1645267237519986444m_-1110853111354515885HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div class="m_-7827408383984974839gmail-m_-1645267237519986444m_-1110853111354515885m_-1455408150298314396gmail_signature"><div dir="ltr"><span style="font-size:12.8px">Clément Béra</span><div style="font-size:12.8px">Pharo consortium engineer</div><div style="font-size:12.8px"><a href="https://clementbera.wordpress.com/" target="_blank">https://clementbera.wordpress.<wbr>com/</a><br></div><div style="font-size:12.8px"><span style="line-height:16px">Bâtiment B 40, avenue Halley 59650 </span><span style="font-weight:bold;line-height:16px">Villeneuve d'Ascq</span></div></div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="m_-7827408383984974839gmail-m_-1645267237519986444m_-1110853111354515885gmail_signature"><div dir="ltr"><span style="font-size:12.8px">Clément Béra</span><div style="font-size:12.8px">Pharo consortium engineer</div><div style="font-size:12.8px"><a href="https://clementbera.wordpress.com/" target="_blank">https://clementbera.wordpress.<wbr>com/</a><br></div><div style="font-size:12.8px"><span style="line-height:16px">Bâtiment B 40, avenue Halley 59650 </span><span style="font-weight:bold;line-height:16px">Villeneuve d'Ascq</span></div></div></div>
</div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="m_-7827408383984974839gmail-m_-1645267237519986444gmail_signature"><div dir="ltr"><span style="font-size:12.8px">Clément Béra</span><div style="font-size:12.8px">Pharo consortium engineer</div><div style="font-size:12.8px"><a href="https://clementbera.wordpress.com/" target="_blank">https://clementbera.wordpress.<wbr>com/</a><br></div><div style="font-size:12.8px"><span style="line-height:16px">Bâtiment B 40, avenue Halley 59650 </span><span style="font-weight:bold;line-height:16px">Villeneuve d'Ascq</span></div></div></div>
</div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="m_-7827408383984974839gmail_signature"><div dir="ltr"><span style="font-size:12.8px">Clément Béra</span><div style="font-size:12.8px">Pharo consortium engineer</div><div style="font-size:12.8px"><a href="https://clementbera.wordpress.com/" target="_blank">https://clementbera.wordpress.<wbr>com/</a><br></div><div style="font-size:12.8px"><span style="line-height:16px">Bâtiment B 40, avenue Halley 59650 </span><span style="font-weight:bold;line-height:16px">Villeneuve d'Ascq</span></div></div></div>
</div></div></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><span style="font-size:12.8px">Clément Béra</span><div style="font-size:12.8px">Pharo consortium engineer</div><div style="font-size:12.8px"><a href="https://clementbera.wordpress.com/" target="_blank">https://clementbera.wordpress.com/</a><br></div><div style="font-size:12.8px"><span style="line-height:16px">Bâtiment B 40, avenue Halley 59650 </span><span style="font-weight:bold;line-height:16px">Villeneuve d'Ascq</span></div></div></div>
</div>