<div dir="ltr">If I understand the code correctly it will do the following:<div><br></div><div>1.8 < 1.8.0 "true"</div><div>1.8.0 < 1.8 "false"</div><div>1.7.4 < 1.8 "true"</div><div>1.8 < 1.7.4 "false"</div><div>1.7.4 < 1.8.0 "true"</div><div>1.8.0 < 1.7.4 "false"</div><div><br></div><div>So concerning 1.8 vs. 1.7.4 the commit message is wrong (typo?), not the implementation.</div><div>For 1.8 vs 1.8.0 your mileage may vary... but I think the new implementation is consistent with the old one in that case. One might add another line to check whether the rest of the other numbers are all 0.<br></div><div><br></div><div>I do not have an image ready on this machine and cannot look into #inSameBranchAs: now. If < previously signalled the error in the 1.8 < 1.7.4 case, this would be a breaking change, depending on how applications rely on it. At work I have software where 1.7 could have been delivered later than 2.0 and may already have features that 2.0 does not have yet (which 2.1 may get and 3.0 will get). While this is not Squeak software, so practically not relevant here, it shows that the change can be debated theoretically.</div><div><br></div><div>Are there test cases for this already? If yes, a test case should be added or one updated for the particular problem that the new commit tries to solve.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Am Mo., 7. Nov. 2022 um 10:21 Uhr schrieb Tobias Pape <<a href="mailto:Das.Linux@gmx.de" target="_blank">Das.Linux@gmx.de</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
> On 7. Nov 2022, at 07:44, Marcel Taeumel <<a href="mailto:marcel.taeumel@hpi.de" target="_blank">marcel.taeumel@hpi.de</a>> wrote:<br>
> <br>
> > e.g. 1.8 is clearly less than 1.7.4<br>
> <br>
> Eh, no it is not? If less means older here ...<br>
<br>
This.<br>
1.8 is exactly the same as 1.8.0, and this sorts definitely _after_ 1.7.4 and is hence, NOT less.<br>
<br>
Best regards<br>
        -Tobias<br>
<br>
> <br>
> Best,<br>
> Marcel<br>
>> Am 05.11.2022 21:36:40 schrieb <a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a> <<a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a>>:<br>
>> <br>
>> Eliot Miranda uploaded a new version of VersionNumber to project The Trunk:<br>
>> <a href="http://source.squeak.org/trunk/VersionNumber-eem.5.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/trunk/VersionNumber-eem.5.mcz</a><br>
>> <br>
>> ==================== Summary ====================<br>
>> <br>
>> Name: VersionNumber-eem.5<br>
>> Author: eem<br>
>> Time: 5 November 2022, 1:36:25.499057 pm<br>
>> UUID: 8883b385-6d1b-4659-b488-c677c700707a<br>
>> Ancestors: VersionNumber-cmm.4<br>
>> <br>
>> Fix VersionNumber>>< so that version numbers of different lengths can be compared. e.g. 1.8 is clearly less than 1.7.4. At the same time fix the comparison so that it terminates as soon as a difference is found.<br>
>> <br>
>> =============== Diff against VersionNumber-cmm.4 ===============<br>
>> <br>
>> Item was changed:<br>
>> ----- Method: VersionNumber>>< (in category 'comparing') -----<br>
>> < another <br>
>> + "Answer whether the receiver is less than the argument, another VersionNumber."<br>
>> - "Answer whether the receiver is less than the argument."<br>
>> <br>
>> + | tmp thisSize thatSize this that |<br>
>> - | tmp |<br>
>> - (self inSameBranchAs: another) ifFalse: <br>
>> - [^self error: 'Receiver and argument in different branches'].<br>
>> - <br>
>> tmp := another numbers.<br>
>> + thisSize := numbers size.<br>
>> + thatSize := tmp size.<br>
>> + 1 to: (thisSize min: thatSize) do:<br>
>> + [:i|<br>
>> + (this := numbers at: i) = (that := tmp at: i) ifFalse:<br>
>> + [^this < that]].<br>
>> + ^thisSize < thatSize!<br>
>> - (tmp size = numbers size) ifTrue:<br>
>> - [1 to: numbers size do: <br>
>> - [ :in | (numbers at: in) < (tmp at: in) ifTrue: [^true]].<br>
>> - ^false].<br>
>> - <br>
>> - ^numbers size < tmp size<br>
>> - !<br>
>> <br>
>> <br>
> <br>
<br>
<br>
<br>
</blockquote></div>