From: mail@...
Date: 2015-02-01T12:41:35+00:00
Subject: [ruby-core:67932] [ruby-trunk - Misc #10809] [PATCH 2/2] Add tests for math.c related to [Feature 10785], [Feature 10800]

Issue #10809 has been updated by kazuki tanaka.


It's my super great pleasure to be able to do that, I'm glad to and go to follow what you said.

But now I'm afraid of committing myself and I realize there'er much things I should know.

So it seems to take time.

Anyway, I wanna to do so. thanks.




----------------------------------------
Misc #10809: [PATCH 2/2] Add tests for math.c related to [Feature 10785], [Feature 10800]
https://bugs.ruby-lang.org/issues/10809#change-51324

* Author: kazuki tanaka
* Status: Open
* Priority: Normal
* Assignee: Nobuyoshi Nakada
----------------------------------------
Hi! Here are two patches.


## reorder_test_cbrt_and_add_a_assertion.patch:
Move Test#test_cbrt to more proper place and add a assertion for Math.cbrt(1.0/0)

## add_test_for_math_c_about_overriding_Integer_to_f.patch:

Add tests for [Feature 10785](https://bugs.ruby-lang.org/issues/10785) and  [Feature 10800](https://bugs.ruby-lang.org/issues/10800)

These two features are merged by Nobuyoshi Nakada san, so I assigned him but please feel free to reassigne.

thanks.

---Files--------------------------------
reorder_test_cbrt_and_add_a_assertion.patch (1.19 KB)
add_test_for_math_c_about_overriding_Integer_to_f.patch (1.36 KB)
use_assert_infinity.patch (2.53 KB)
revert_to_original_methods.patch (1.1 KB)


-- 
https://bugs.ruby-lang.org/