<div dir="ltr">Hi Nicolas,<div><br></div><div>    I think you should move this to trunk.</div></div><br><div class="gmail_quote"><div dir="ltr">On Sun, Dec 9, 2018 at 2:17 AM <<a href="mailto:commits@source.squeak.org">commits@source.squeak.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Nicolas Cellier uploaded a new version of Kernel to project The Inbox:<br>
<a href="http://source.squeak.org/inbox/Kernel-nice.1200.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/inbox/Kernel-nice.1200.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: Kernel-nice.1200<br>
Author: nice<br>
Time: 9 December 2018, 11:17:42.830548 am<br>
UUID: 4608ee87-5859-4132-918f-df0f71dcc07e<br>
Ancestors: Kernel-eem.1199<br>
<br>
Let's make basicInspect robust to crazy numbers.<br>
<br>
With <a href="http://www.squeaksource.com/ArbitraryPrecisionFl.html" rel="noreferrer" target="_blank">http://www.squeaksource.com/ArbitraryPrecisionFl.html</a><br>
I can create crazy numbers like this:<br>
((1 asArbitraryPrecisionFloatNumBits: 100) timesTwoPower: 1e100)<br>
<br>
But then I can't print to decimal form (a bit too many bytes to be allocated....).<br>
<br>
The problem when debugging such monster is that basicInspect is not basicEnough: it insists on printing the Number in the inspector title (label)!<br>
<br>
If we invoke basicInspect, we generally want something basic. Even if we just inspect, having a label spanning more than 100 chars is vain...<br>
<br>
So let's be robust.<br>
<br>
=============== Diff against Kernel-eem.1199 ===============<br>
<br>
Item was changed:<br>
  ----- Method: Number>>defaultLabelForInspector (in category 'printing') -----<br>
  defaultLabelForInspector<br>
        "Answer the default label to be used for an Inspector window on the receiver."<br>
<br>
+       ^ super defaultLabelForInspector, ': ', ([self printStringLimitedTo: 100]<br>
+               ifError: ['<error cannot print this number>'])!<br>
-       ^ super defaultLabelForInspector, ': ', self printString!<br>
<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div>