<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<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>No need to apologize, this was a nice debugging exercise. :D</p>
<p><br>
</p>
<p>Best,</p>
<p>Christoph</p>
</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 David T. Lewis <lewis@mail.msen.com><br>
<b>Gesendet:</b> Freitag, 17. Dezember 2021 04:14:46<br>
<b>An:</b> squeak-dev@lists.squeakfoundation.org<br>
<b>Betreff:</b> Re: [squeak-dev] The Trunk: System-ct.1269.mcz</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">Thanks Christoph and Jaromir for finding and fixing this, and I<br>
sincerely apologize for introducing the change without testing it<br>
properly with #processPreemptionYields.<br>
<br>
Thank you!<br>
Dave<br>
<br>
<br>
On Fri, Dec 17, 2021 at 12:23:08AM +0000, commits@source.squeak.org wrote:<br>
> Christoph Thiede uploaded a new version of System to project The Trunk:<br>
> <a href="http://source.squeak.org/trunk/System-ct.1269.mcz">http://source.squeak.org/trunk/System-ct.1269.mcz</a><br>
> <br>
> ==================== Summary ====================<br>
> <br>
> Name: System-ct.1269<br>
> Author: ct<br>
> Time: 17 December 2021, 1:23:02.870987 am<br>
> UUID: 6e5ae88f-54c6-0445-8ec4-9f0005e2a14a<br>
> Ancestors: System-ct.1268<br>
> <br>
> Revises SmalltalkImage >> #vmParameterAt:ifAbsent:. Avoids running defaultValueOrBlock inside the internal error handler.<br>
> <br>
> Thanks to Jaromir (jar) for the warning!<br>
> <br>
> =============== Diff against System-ct.1268 ===============<br>
> <br>
> Item was changed:<br>
>   ----- Method: SmalltalkImage>>vmParameterAt:ifAbsent: (in category 'vm parameters') -----<br>
>   vmParameterAt: parameterIndex ifAbsent: defaultValueOrBlock<br>
> +      "Answer a VM parameter or defaultValueOrBlock value if out of range or if the VM does not provide a value for this parameter. A VM parameter is typically numeric or boolean, and if not implemented will be nil."<br>
> +      ^ ([self vmParameterAt: parameterIndex]<br>
> +              on: Error do: [])<br>
> +                      ifNil: [defaultValueOrBlock value]!<br>
> -      "Answer a VM parameter or defaultValueOrBlock value if out of range<br>
> -      or if the VM does not provide a value for this parameter. A VM parameter<br>
> -      is typically numeric or boolean, and if not implemented will be nil."<br>
> -      ^ [(self vmParameterAt: parameterIndex)<br>
> -                      ifNil: [^ defaultValueOrBlock value]]<br>
> -              on: Error<br>
> -              do: [defaultValueOrBlock value]!<br>
> <br>
> <br>
<br>
</div>
</span></font>
</body>
</html>