Mantis Bugtracker
  

Viewing Issue Simple Details Jump to Notes ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0006779 [Squeak] Kernel minor always 11-22-07 21:21 04-18-10 22:05
Reporter nicolas cellier View Status public  
Assigned To andreas
Priority normal Resolution fixed  
Status closed   Product Version 3.10
Summary 0006779: ScaledDecimal readFrom: '5.3' would not answer a ScaledDecimal
Description
It inherits this method from Number and Answer a Float...
Adding a posteriori asScaledDecimal conversion
- is not easy (How many digits?)
- is not good, passing through a Float will introduce roundOff errors

It is better hacking SqNumberParser
Additional Information
Attached Files  ScaledDecimal-readFrom-M6779-nice-Patch.1.cs [^] (2,473 bytes) 11-22-07 21:35
 ScaledDecimal-readFrom-M6779-nice-Test.1.cs [^] (795 bytes) 11-22-07 21:36
 ScaledDecimal-readFrom-M6779-nice-Patch.2.cs [^] (2,626 bytes) 08-29-08 20:27

- Relationships

- Notes
(0011563 - 195 - 261 - 261 - 261 - 261 - 261)
Keith_Hodges
12-17-07 17:03
edited on: 12-17-07 17:10

"fix begin"
Installer mantis bug: 6779 fix:'ScaledDecimal-readFrom-M6779-nice-Patch.1.cs'.
"fix test"
Installer mantis bug: 6779 fix:'ScaledDecimal-readFrom-M6779-nice-Test.1.cs'.
"fix end"

 
(0012542 - 230 - 302 - 302 - 302 - 302 - 302)
nicolas cellier
08-29-08 20:11
edited on: 08-29-08 20:27

"fix begin"
Installer mantis ensureFix: 7169.
Installer mantis bug: 6779 fix:'ScaledDecimal-readFrom-M6779-nice-Patch.2.cs'.
"fix test"
Installer mantis bug: 6779 fix:'ScaledDecimal-readFrom-M6779-nice-Test.1.cs'.
"fix end"

 
(0012544 - 190 - 238 - 238 - 363 - 363 - 363)
nicolas cellier
08-29-08 20:30
edited on: 08-29-08 20:30

Patch number 1 is bugged as every SqNumberParser method for reading ScaledDecimal:

self assert: (ScaledDecimal readFrom: '0.50') = (1/2)

Patch number 2 correct this

See also 0007169

 
(0013279 - 59 - 59 - 209 - 209 - 209 - 209)
nicolas cellier
08-24-09 20:17

Fixed in http://source.squeak.org/trunk/Kernel-nice.202.mcz [^]
 

- Issue History
Date Modified Username Field Change
11-22-07 21:21 nicolas cellier New Issue
11-22-07 21:35 nicolas cellier File Added: ScaledDecimal-readFrom-M6779-nice-Patch.1.cs
11-22-07 21:36 nicolas cellier File Added: ScaledDecimal-readFrom-M6779-nice-Test.1.cs
12-17-07 17:03 Keith_Hodges Note Added: 0011563
12-17-07 17:10 Keith_Hodges Note Edited: 0011563
08-29-08 20:11 nicolas cellier Note Added: 0012542
08-29-08 20:27 nicolas cellier File Added: ScaledDecimal-readFrom-M6779-nice-Patch.2.cs
08-29-08 20:27 nicolas cellier Note Edited: 0012542
08-29-08 20:30 nicolas cellier Note Added: 0012544
08-29-08 20:30 nicolas cellier Note Edited: 0012544
01-10-09 02:00 Keith_Hodges Status new => pending
01-10-09 03:29 Keith_Hodges Status pending => testing
01-10-09 03:39 Keith_Hodges Status testing => resolved
01-10-09 03:39 Keith_Hodges Fixed in Version  => 3.11
01-10-09 03:39 Keith_Hodges Resolution open => fixed
01-10-09 03:39 Keith_Hodges Assigned To  => Keith_Hodges
01-10-09 03:41 Keith_Hodges Status resolved => testing
08-24-09 20:17 nicolas cellier Note Added: 0013279
10-03-09 19:33 Keith_Hodges Status testing => assigned
10-03-09 19:33 Keith_Hodges Assigned To Keith_Hodges => andreas
10-03-09 20:14 nicolas cellier Status assigned => resolved
10-03-09 20:14 nicolas cellier Fixed in Version 3.11 => trunk
04-18-10 22:05 andreas Status resolved => closed


Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
73 total queries executed.
39 unique queries executed.
Powered by Mantis Bugtracker