Mantis Bugtracker
  

Viewing Issue Advanced Details Jump to Notes ] View Simple ] 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 Platform
Status closed   OS
Projection none   OS Version
ETA none Fixed in Version trunk Product Version 3.10
  Product Build
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
Steps To Reproduce
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