[Mono-bugs] [Bug 60110][Cos] Changed - Double.ToString with format R is different (longer) than MS version

bugzilla-daemon@bugzilla.ximian.com bugzilla-daemon@bugzilla.ximian.com
Wed, 16 Jun 2004 15:31:01 -0400 (EDT)


Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by sebastien@ximian.com.

http://bugzilla.ximian.com/show_bug.cgi?id=60110

--- shadow/60110	2004-06-16 12:08:00.000000000 -0400
+++ shadow/60110.tmp.15147	2004-06-16 15:31:01.000000000 -0400
@@ -3,21 +3,21 @@
 Version: unspecified
 OS: unknown
 OS Details: 
 Status: NEW   
 Resolution: 
 Severity: Unknown
-Priority: Normal
+Priority: Cosmetic
 Component: CORLIB
 AssignedTo: mono-bugs@ximian.com                            
 ReportedBy: gert.driesen@pandora.be               
 QAContact: mono-bugs@ximian.com
 TargetMilestone: ---
 URL: 
 Cc: 
-Summary: Double.ToString with format R does not roundtrip
+Summary: Double.ToString with format R is different (longer) than MS version
 
 The Mono roundtrip string format of a double doesn't roundtrip.
 
 Execute the following code :
 
 using System;
@@ -52,6 +52,22 @@
 
 The current design (Normalize in FloatingPointFormatter) cannot be
 used to implement "R" because it loose precision when doing
 multiplication (so we end up with the right number of decimals but not
 with the proper numbers).
 
+
+------- Additional Comments From sebastien@ximian.com  2004-06-16 15:31 -------
+This is "somewhat" fixed in CVS thanks to source code from Jon Skeet
+(http://www.yoda.arachsys.com/). 
+
+I say "somewhat" because roundtrip is possible (with full precision)
+but the "R" output isn't exactly like Microsoft (it's much longer).
+The good enws is that MS doesn't seems to have problem when reading
+our long-R and Mono accept the short-R version.
+
+I added two new unit tests to track this. The first one tracks our
+different "R" output. The second verifies that both Mono and MS output
+are compatible on both runtime.
+
+I'm changing the bug summary to match the remaining problem and
+downgrade priority to "cosmetic".