<div dir="ltr">Hi K, Hi David,<div><br></div><div>    can you explain this to me?  I don't understand why one wouldn't want to test that the 64-bit Cog image format is valid.  This probably just needs a few more words.  I'm not suggesting that the change is wrong, simply that I don't understand it.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 10, 2019 at 2:22 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-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"> <br>
David T. Lewis uploaded a new version of ImageFormat to project VM Maker:<br>
<a href="http://source.squeak.org/VMMaker/ImageFormat-kks.35.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/VMMaker/ImageFormat-kks.35.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: ImageFormat-kks.35<br>
Author: kks<br>
Time: 10 April 2019, 11:21:27.422478 pm<br>
UUID: 67c4b2d3-702b-447e-890c-1ba698a71b4f<br>
Ancestors: ImageFormat-kks.34<br>
<br>
Removed tests for 68004, 68021 formats since 68004 is no longer a base version number.<br>
<br>
=============== Diff against ImageFormat-kks.34 ===============<br>
<br>
Item was changed:<br>
  ----- Method: ImageFormatTest>>testIsValidVersionNumber (in category 'testing') -----<br>
  testIsValidVersionNumber<br>
<br>
        self should: [ImageFormat fromInteger: 0] raise: Error.<br>
        self should: [ImageFormat fromInteger: (6502 bitAnd: 16r80000000)] raise: Error.<br>
        self should: [ImageFormat fromInteger: (6502 bitAt: 31 put: 1)] raise: Error.<br>
        self should: [ImageFormat fromInteger: 6500] raise: Error.<br>
        self should: [ImageFormat fromInteger: 6501] raise: Error.<br>
        self should: [ImageFormat fromInteger: 6503] raise: Error. "Cog requires both capabilities"<br>
        self should: [ImageFormat fromInteger: 68001] raise: Error. "Cog requires both capabilities"<br>
<br>
        self assert: ImageFormat default isValidVersionNumber.<br>
        self assert: (ImageFormat wordSize: 4 closures: false) isValidVersionNumber.<br>
        self assert: (ImageFormat wordSize: 4 closures: true) isValidVersionNumber.<br>
        self assert: (ImageFormat wordSize: 8 closures: false) isValidVersionNumber.<br>
        self assert: (ImageFormat wordSize: 8 closures: true) isValidVersionNumber.<br>
        self assert: (ImageFormat fromInteger: 6502) isValidVersionNumber.<br>
        self assert: (ImageFormat fromInteger: (6502 bitAt: 31 put: 0)) isValidVersionNumber.<br>
        self assert: (ImageFormat fromInteger: 6521) isValidVersionNumber.<br>
        self assert: (ImageFormat fromInteger:68000) isValidVersionNumber.<br>
        self assert: (ImageFormat fromInteger:68002) isValidVersionNumber.<br>
-       self assert: (ImageFormat fromInteger:68004) isValidVersionNumber.<br>
        self assert: (ImageFormat fromInteger:68003) isValidVersionNumber. "valid but unused, as with 68019"<br>
        self assert: (ImageFormat fromInteger: 68019) isValidVersionNumber.<br>
-       self assert: (ImageFormat fromInteger: 68021) isValidVersionNumber.<br>
<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>