Re: [scheme-requests-for-implementation/srfi-48] Fix test for Chicken (#12)
hamay1010 14 Nov 2020 04:23 UTC
- The output of format ~F is dependent on number->string .
```
;; Chicken 5.2.0
(number->string 3.4569e15) ; ==> "3456900000000000.0"
(number->string 3.456e15) ; ==> "3456000000000000.0"
;; Chicken 4.12.0
(number->string 3.4569e15) ; ==> "3.4569e+15"
(number->string 3.456e15) ; ==> "3.456e+15"
;; Guile 2.2.7
(number->string 3.4569e15) ; ==> "3.4569e15"
(number->string 3.456e15) ; ==> "3.456e15"
;; Gauche 0.9.9
(number->string 3.4569e15) ; ==> "3.4569e15"
(number->string 3.456e15) ; ==> "3.456e15"
```
- Sorry, I don't remember why the old test-0001-Chicken.scm used
zero-padded exponent notation (e.g. "3.457e+015" ) .
It might be something copy miss ( test-0001-Racket.scm uses "3.457e+015" ) .
2020年11月14日(土) 3:37 Arthur A. Gleckler <xxxxxx@speechcode.com>:
>
> CC-ing Ken Dickey.
>
> Thank you for your contribution. However, I don't understand. It looks like this change simply changes the results to match what Chicken is doing, but shouldn't Chicken be giving the same results as other implementations? On the other hand, I don't understand why the results used to zero-pad the exponent, and it looks like this change is making the Chicken results match the results for other implementations. Is that right?
>
> Thanks for the clarification.
>
> On Fri, Nov 13, 2020 at 10:13 AM Hamayama <xxxxxx@github.com> wrote:
>>
>> I fixed test for Chicken.
>>
>> mailing list:
>> https://srfi-email.schemers.org/srfi-48/msg/15509362/
>>
>> issue:
>> diamond-lizard/srfi-48#1
>>
>> pull request:
>> diamond-lizard/srfi-48#2
>>
>> ________________________________
>>
>> You can view, comment on, or merge this pull request online at:
>>
>> https://github.com/scheme-requests-for-implementation/srfi-48/pull/12
>>
>> Commit Summary
>>
>> Fix test for Chicken
>>
>> File Changes
>>
>> M test/test-0001-Chicken.scm (76)
>>
>> Patch Links:
>>
>> https://github.com/scheme-requests-for-implementation/srfi-48/pull/12.patch
>> https://github.com/scheme-requests-for-implementation/srfi-48/pull/12.diff
>>
>> —