egallesio / STklos

STklos Scheme
http://stklos.net
GNU General Public License v2.0
69 stars 17 forks source link

Be more careful when testing NaNs (and fix a failing test on Android). #575

Closed jpellegrini closed 1 year ago

jpellegrini commented 1 year ago

(* 0+1i +inf.0) can be -nan.0+inf.0i (on Linux/amd64 for example) and also +nan.0+inf.0i (on Android, for example). So we don't test the sign of the NaN.

jpellegrini commented 1 year ago

(This morning I realized that this test was failing on Android)

jpellegrini commented 1 year ago

@egallesio I think this is not necessary anymore, since you have committed a much better fix! :)

By the way, all tests pass on Android now. :smiley:

egallesio commented 1 year ago

Oops. Forgot to tell you that it is corrected now, but you have seen it :smiley: