<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<meta content="text/html; charset=UTF-8">
<style type="text/css" style="">
<!--
p
        {margin-top:0;
        margin-bottom:0}
-->
</style>
<div dir="ltr">
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Helvetica,sans-serif">
<p>(General note: I think we should use #flag: for code like this. This seems to be the most likely chance to me that any further volunteer will find and resolve the defect if we forget it. :-))</p>
<div id="x_Signature">
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:rgb(0,0,0); font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols">
<div name="x_divtagdefaultwrapper" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:; margin:0">
<div><font size="2" color="#808080"></font></div>
</div>
</div>
</div>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>Von:</b> Squeak-dev <squeak-dev-bounces@lists.squeakfoundation.org> im Auftrag von Nicolas Cellier <nicolas.cellier.aka.nice@gmail.com><br>
<b>Gesendet:</b> Donnerstag, 29. April 2021 22:53:49<br>
<b>An:</b> The general-purpose Squeak developers list<br>
<b>Betreff:</b> Re: [squeak-dev] The Trunk: Tests-nice.459.mcz</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">Hmm no, this does not solve anything.<br>
It happened to work once or twice, but no more...<br>
It's impossible to debug, some doesNotUnderstand: occurs that install<br>
and set state to #inactive, but debugger interacts and is causing more<br>
doesNotUnderstand:<br>
<br>
Le jeu. 29 avr. 2021 à 22:30, <commits@source.squeak.org> a écrit :<br>
><br>
> Nicolas Cellier uploaded a new version of Tests to project The Trunk:<br>
> <a href="http://source.squeak.org/trunk/Tests-nice.459.mcz">http://source.squeak.org/trunk/Tests-nice.459.mcz</a><br>
><br>
> ==================== Summary ====================<br>
><br>
> Name: Tests-nice.459<br>
> Author: nice<br>
> Time: 29 April 2021, 10:30:03.551961 pm<br>
> UUID: 67799656-9640-e447-b665-6f74573e3716<br>
> Ancestors: Tests-nice.458<br>
><br>
> For some reason, ImageSegmentTest>>testContextsShouldBeWritableToaFile interact badly with progress bar, resulting in a context that cannotReturn:<br>
> Workaround by temporarily suppressing progress.<br>
><br>
> =============== Diff against Tests-nice.458 ===============<br>
><br>
> Item was changed:<br>
>   ----- Method: ImageSegmentTest>>testContextsShouldBeWritableToaFile (in category 'tests') -----<br>
>   testContextsShouldBeWritableToaFile<br>
>         "This should not throw an exception"<br>
> +       [NativeImageSegment new<br>
> -       NativeImageSegment new<br>
>                 copyFromRoots: {thisContext. thisContext copyStack} sizeHint: 100;<br>
>                 extract;<br>
>                 writeToFile: 'ContextChain';<br>
> +               yourself] on: ProgressInitiationException do: [:e | e resumeSuppressingProgress]<br>
> -               yourself<br>
><br>
>         "TODO: write assertions showing that something meaningful actually happened."<br>
><br>
>         "TODO:  bring them back in again"!<br>
><br>
><br>
<br>
</div>
</span></font>
</body>
</html>