Skip to content

Commit

Permalink
Use "fixed" structure Real64 with recent versions of SML/NJ
Browse files Browse the repository at this point in the history
Closes MLton#523

Although SML/NJ 110.99.3 fixed some issues with `structure Real{,64}`
failing to match the SML Basis Library specification, it did not
implement `Real.fmt StringCvt.EXACT`.  So, a "fixed" version of
`structure Real64` needs to be exported (rather than exposing the
version of `structure Real64` from `$/basis.cm`) and it is simplest to
revert back to `structure Real64 = Real`
(where `structure Real = FixReal(struct open Pervasive.Real end)`).
  • Loading branch information
MatthewFluet committed Nov 7, 2023
1 parent 38eb9de commit b9cebc6
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions lib/stubs/smlnj/basis/sources.cm
Original file line number Diff line number Diff line change
Expand Up @@ -85,9 +85,7 @@ real.0.sml
real.1.sml
#endif
../../common/basis/real32-via-real.sml
#if SMLNJ_VERSION < 110 orelse (SMLNJ_VERSION = 110 andalso SMLNJ_MINOR_VERSION < 99) orelse (SMLNJ_VERSION = 110 andalso SMLNJ_MINOR_VERSION = 99 andalso SMLNJ_PATCH_VERSION < 3)
../../common/basis/real64-via-real.sml
#endif

#if SMLNJ_VERSION < 110 orelse (SMLNJ_VERSION = 110 andalso SMLNJ_MINOR_VERSION < 91)
../../common/basis/pack-word64-dummy.sml
Expand Down

0 comments on commit b9cebc6

Please sign in to comment.