If I understand the code correctly it will do the following:

1.8 < 1.8.0 "true"
1.8.0 < 1.8 "false"
1.7.4 < 1.8 "true"
1.8 < 1.7.4 "false"
1.7.4 < 1.8.0 "true"
1.8.0 < 1.7.4 "false"

So concerning 1.8 vs. 1.7.4 the commit message is wrong (typo?), not the
implementation.
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.

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.

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.


Am Mo., 7. Nov. 2022 um 10:21 Uhr schrieb Tobias Pape <Das.Linux at gmx.de>:

>
>
> > On 7. Nov 2022, at 07:44, Marcel Taeumel <marcel.taeumel at hpi.de> wrote:
> >
> > > e.g. 1.8 is clearly less than 1.7.4
> >
> > Eh, no it is not? If less means older here ...
>
> This.
> 1.8 is exactly the same as 1.8.0, and this sorts definitely _after_ 1.7.4
> and is hence, NOT less.
>
> Best regards
>         -Tobias
>
> >
> > Best,
> > Marcel
> >> Am 05.11.2022 21:36:40 schrieb commits at source.squeak.org <
> commits at source.squeak.org>:
> >>
> >> Eliot Miranda uploaded a new version of VersionNumber to project The
> Trunk:
> >> http://source.squeak.org/trunk/VersionNumber-eem.5.mcz
> >>
> >> ==================== Summary ====================
> >>
> >> Name: VersionNumber-eem.5
> >> Author: eem
> >> Time: 5 November 2022, 1:36:25.499057 pm
> >> UUID: 8883b385-6d1b-4659-b488-c677c700707a
> >> Ancestors: VersionNumber-cmm.4
> >>
> >> 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.
> >>
> >> =============== Diff against VersionNumber-cmm.4 ===============
> >>
> >> Item was changed:
> >> ----- Method: VersionNumber>>< (in category 'comparing') -----
> >> < another
> >> + "Answer whether the receiver is less than the argument, another
> VersionNumber."
> >> - "Answer whether the receiver is less than the argument."
> >>
> >> + | tmp thisSize thatSize this that |
> >> - | tmp |
> >> - (self inSameBranchAs: another) ifFalse:
> >> - [^self error: 'Receiver and argument in different branches'].
> >> -
> >> tmp := another numbers.
> >> + thisSize := numbers size.
> >> + thatSize := tmp size.
> >> + 1 to: (thisSize min: thatSize) do:
> >> + [:i|
> >> + (this := numbers at: i) = (that := tmp at: i) ifFalse:
> >> + [^this < that]].
> >> + ^thisSize < thatSize!
> >> - (tmp size = numbers size) ifTrue:
> >> - [1 to: numbers size do:
> >> - [ :in | (numbers at: in) < (tmp at: in) ifTrue: [^true]].
> >> - ^false].
> >> -
> >> - ^numbers size < tmp size
> >> - !
> >>
> >>
> >
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20221107/3faff2b3/attachment.html>