[BUG][FIX] Interval method includes:
Boris Gaertner
Boris.Gaertner at gmx.net
Tue Jun 15 14:22:14 UTC 2004
This bug report is for Squeak 3.7beta latest update 5954
This is NOT a dramatic bug
The inclusion test does not work for intervals with a negative
step. To see the problem, you can try this:
(-1 to: -111 by: -1) includes: -100
This answers false.
To find the element, you can evaluate this statement:
(-1 to: -111 by: -1) indexOf: -100
This answers 100, which is obviously the right answer.
The test
(-1 to: -111 by: -1) at: 100
confirms this.
The bug is in method valuesInclude: where we use the signed step,
scale it and compare it with an absolute value. We have to use the
absolute value of the step, too.
Greetings,
Boris
-------------- next part --------------
A non-text attachment was scrubbed...
Name: IntervalInclusionFix.2.cs
Type: application/octet-stream
Size: 734 bytes
Desc: not available
Url : http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20040615/9eadcb30/IntervalInclusionFix.2.obj
More information about the Squeak-dev
mailing list
|