0007040: #asFloatSimply should be in LargePositiveInteger

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

Category

Severity

Reproducibility

Date Submitted

Last Update

0007040

[Squeak] Kernel

minor

always

05-10-08 14:22

09-19-09 01:54

Reporter

hfm

View Status

public

Assigned To

nicolas cellier

Priority

normal

Resolution

fixed

Status

resolved

Product Version

Summary

0007040: #asFloatSimply should be in LargePositiveInteger

Description

Sending

"aSmallInteger" 5 asFloatSimply

gives
Error: Instances of SmallInteger are not indexable.
However it does work fine for *LargeInteger:

4585759856567 asFloatSimply

Fix is attached.

Additional Information

Attached Files

Integer#asFloatSimply.1.cs [^] (809 bytes) 05-10-08 14:22

Relationships

Notes
(0012077 - 253 - 283 - 283 - 283 - 283 - 283) nicolas cellier 05-10-08 15:15	Oh yes, you are right. asFloatSimply was part of an initial patch for converting (large) integers to nearest Float, and used to be called from LargeInteger only. But it has soon been superseded. IMO, the best thing is to remove it from the image.

(0013206 - 61 - 73 - 223 - 223 - 223 - 223) nicolas cellier 07-19-09 19:58 edited on: 07-19-09 19:59	Fixed in http://source.squeak.org/trunk/Kernel-nice.196.mcz [^]

Issue History
Date Modified	Username	Field	Change
05-10-08 14:22	hfm	New Issue
05-10-08 14:22	hfm	File Added: Integer#asFloatSimply.1.cs
05-10-08 15:15	nicolas cellier	Note Added: 0012077
07-19-09 19:58	nicolas cellier	Note Added: 0013206
07-19-09 19:59	nicolas cellier	Note Edited: 0013206
09-19-09 01:54	nicolas cellier	Status	new => resolved
09-19-09 01:54	nicolas cellier	Fixed in Version	=> trunk
09-19-09 01:54	nicolas cellier	Resolution	open => fixed
09-19-09 01:54	nicolas cellier	Assigned To	=> nicolas cellier

Mantis 1.0.8[^]

45 total queries executed.
33 unique queries executed.