Feature #13508
closedHow remove/refactor code related mathn library.
Description
I removed mathn library at r58413 and r58432.
I have some concerns related mathn library. This issue is reminder for this concern.
- Still remains conditions for mathn loaded. ex. https://github.com/ruby/ruby/blob/trunk/array.c#L5755
- {complex,rational}.c have {nucomp,nurat}_canonicalization methods for mathn.
- lib/cmath.rb have workaround code for mathn see. https://github.com/ruby/ruby/blob/trunk/lib/cmath.rb#L27
I continue to discuss concerns to @mrkn (Kenta Murata)
Updated by shyouhei (Shyouhei Urabe) about 9 years ago
- Status changed from Open to Assigned
Updated by hsbt (Hiroshi SHIBATA) over 8 years ago
- Target version changed from 2.5 to 2.6
I gave up to apply at Ruby 2.5. I try this next year.
Updated by naruse (Yui NARUSE) over 8 years ago
- Target version deleted (
2.6)
Updated by hsbt (Hiroshi SHIBATA) 3 days ago
- Status changed from Assigned to Closed
All the items in this ticket have been resolved.
The mathn-specific code in array.c, complex.c, rational.c and enum.c was removed at https://github.com/ruby/ruby/commit/b958e2add835d62c0a62edaf9a23ecbbd70a3635 (https://github.com/ruby/ruby/pull/3691), and the obsolete --enable-mathn configure option was removed at https://github.com/ruby/ruby/commit/89110b27437d4a84f7cb0387e59fd40d94efee1d.
lib/cmath.rb was removed from the ruby repository at https://github.com/ruby/ruby/commit/a3b85016149f4f73f08c739626b2f02badd5ad17. The mathn workaround remaining in the cmath gem is intentionally kept for compatibility with the mathn gem.