Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#822 closed defect (fixed)

numbers-string-conversion-test fails for windows

Reported by: felix winkelmann Owned by: felix winkelmann
Priority: major Milestone: 4.9.0
Component: core libraries Version: 4.7.x
Keywords: Cc:
Estimated difficulty:

Description

One case from the numbers-string-conversion test fails:

diff --git a/tests/numbers-string-conversion-tests.scm b/tests/numbers-string-conversion-tests.scm
index 09d8ee8..7abfd9d 100644
--- a/tests/numbers-string-conversion-tests.scm
+++ b/tests/numbers-string-conversion-tests.scm
@@ -209,7 +209,8 @@

("-1/2" (- (/ 1 2)) "-0.5" "-.5" "-500.0e-3")
("1/-2" #f)
("10/0" +inf.0 "+inf.0")

  • ("0/10" 0.0 "0.0")

+;XXX fails on Windows
+; ("0/10" 0.0 "0.0")

("#e0/10" 0 "0")
("#e1#/2" 5 "5")
("#e1/2#" #f)

Can I just add a "0." to the result list in the line above?

Change History (3)

comment:1 Changed 10 years ago by sjamaan

Owner: changed from sjamaan to felix winkelmann
Status: newassigned

Sure, that's acceptable since "0." is valid numeric syntax, I just didn't think about adding it. you might also add ".0" just in case :)

comment:2 Changed 10 years ago by felix winkelmann

Resolution: fixed
Status: assignedclosed

I have added an entry and pushed the changes.

comment:3 Changed 10 years ago by felix winkelmann

Milestone: 4.8.04.9.0

Milestone 4.8.0 deleted

Note: See TracTickets for help on using tickets.