From: David Smith Date: 2015-02-01T14:15:12+11:00 Subject: [ruby-core:67922] unsubscribe --54cd9a4f_327b23c6_5a7 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline --=C2=A0 David Smith djs070=40gmail.com 0408 692 261 On 1 =46ebruary 2015 at 2:14:46 pm, nobu=40ruby-lang.org (nobu=40ruby-lan= g.org) wrote: Issue =2310809 has been updated by Nobuyoshi Nakada. =20 > =7E=7E=7Ediff =20 > + assert=5Fequal(1.0/0, Math.cbrt(1.0/0)) =20 > =7E=7E=7E =20 Isn't it better to use =60assert=5Finfinity=60=3F =20 > =7E=7E=7Ediff =20 > + def to=5Ff =20 > + =5Fto=5Ff =20 > + end =20 > =7E=7E=7E =20 Why not revert to the original methods=3F =20 ---------------------------------------- =20 Misc =2310809: =5BPATCH 2/2=5D Add tests for math.c related to =5B=46eatu= re 10785=5D, =5B=46eature 10800=5D =20 https://bugs.ruby-lang.org/issues/10809=23change-51315 =20 * Author: kazuki tanaka =20 * Status: Open =20 * Priority: Normal =20 * Assignee: Nobuyoshi Nakada =20 ---------------------------------------- =20 Hi=21 Here are two patches. =20 =23=23 reorder=5Ftest=5Fcbrt=5Fand=5Fadd=5Fa=5Fassertion.patch: =20 Move Test=23test=5Fcbrt to more proper place and add a assertion for Math= .cbrt(1.0/0) =20 =23=23 add=5Ftest=5Ffor=5Fmath=5Fc=5Fabout=5Foverriding=5FInteger=5Fto=5F= f.patch: =20 Add tests for =5B=46eature 10785=5D(https://bugs.ruby-lang.org/issues/107= 85) and =5B=46eature 10800=5D(https://bugs.ruby-lang.org/issues/10800) =20 These two features are merged by Nobuyoshi Nakada san, so I assigned him = but please feel free to reassigne. =20 thanks. =20 ---=46iles-------------------------------- =20 reorder=5Ftest=5Fcbrt=5Fand=5Fadd=5Fa=5Fassertion.patch (1.19 KB) =20 add=5Ftest=5Ffor=5Fmath=5Fc=5Fabout=5Foverriding=5FInteger=5Fto=5Ff.patch= (1.36 KB) =20 -- =20 https://bugs.ruby-lang.org/ =20 --54cd9a4f_327b23c6_5a7 Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline