Re: [scheme-requests-for-implementation/srfi-48] Fix test for Chicken (#12) hamay1010 (14 Nov 2020 04:24 UTC)

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
>>
>> —