<div dir="ltr"><div dir="ltr"><div>I just fixed a typo that went unnoticed for ~9 years (see <span style="color:rgb(0,0,0);white-space:pre-wrap">KernelTests-codefrau.395). Even when Levente revised the code in 2015, he did not spot it:</span></div><div><br></div></div><div dir="ltr"><img src="cid:ii_kn25hux10" alt="image.png" width="542" height="226"><br></div><div><br></div><div><div dir="ltr"><div><span style="color:rgb(0,0,0);white-space:pre-wrap">IMHO (speaking as a typo nerd) this is due to our default code font not having distinct enough letter shapes. I very much like our use of a proportional font, but maybe it's time to switch to a different one?</span></div><div></div></div><div dir="ltr"></div></div><div><br></div>- Vanessa -<div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Apr 3, 2021 at 12:06 PM <<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:1px solid rgb(204,204,204);padding-left:1ex">Vanessa Freudenberg uploaded a new version of KernelTests to project The Trunk:<br>
<a href="http://source.squeak.org/trunk/KernelTests-codefrau.395.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/trunk/KernelTests-codefrau.395.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: KernelTests-codefrau.395<br>
Author: codefrau<br>
Time: 3 April 2021, 12:06:03.17026 pm<br>
UUID: 062e8b73-a62e-46ce-af80-0dbb34c2f8dc<br>
Ancestors: KernelTests-mt.394<br>
<br>
Fix typo in CompiledMethodComparisonTest.<br>
<br>
=============== Diff against KernelTests-mt.394 ===============<br>
<br>
Item was changed:<br>
  ----- Method: CompiledMethodComparisonTest>>testHash (in category 'tests') -----<br>
  testHash<br>
        <timeout: 120><br>
<br>
        | ai |<br>
        ai := CompiledMethod allInstances.<br>
        "We assume here that if two CompiledMethods are equal then they have the same size and header."<br>
        (ai groupBy: [ :method | { method size. method header } ]) values<br>
                replace: [ :each | each asArray ];<br>
                do: [ :methods |<br>
                        1 to: methods size do: [ :i |<br>
                                i to: methods size do: [ :j |<br>
                                        (methods at: i) = (methods at: j) ifTrue: [<br>
+                                               self assert: (methods at: i) hash equals: (methods at: j) hash ] ] ] ]<br>
-                                               self assert: (methods at: i) hash equals: (methods at: i) hash ] ] ] ]<br>
                displayingProgress: 'Testing hashes'.<br>
        self assert: (ai collect: [ :cm | cm hash ] as: Set) size * 2 >= ai asSet size!<br>
<br>
<br>
</blockquote></div></div></div>