From: ngotogenome@...
Date: 2015-11-20T14:27:28+00:00
Subject: [ruby-core:71608] [Ruby trunk - Feature #10974] [PATCH] Remove methods which has suffix `!`(sin!, cos!…) from CMath

Issue #10974 has been updated by Naohisa Goto.


The infinite loop observed during RubySpec test in r52469
(http://rubyci.s3.amazonaws.com/ubuntu1510/ruby-trunk/log/20151106T153002Z.fail.html.gz )
is caused by the following code in mathn.rb.

~~~
unless defined?(Math.exp!)
  Object.instance_eval{remove_const :Math}
  Math = CMath # :nodoc:
end
~~~


----------------------------------------
Feature #10974: [PATCH] Remove methods which has suffix `!`(sin!, cos!���) from CMath
https://bugs.ruby-lang.org/issues/10974#change-54992

* Author: gogo tanaka
* Status: Open
* Priority: Normal
* Assignee: 
----------------------------------------
Hi, now I���d like to remove methods with `!`( `sin!`, `cos!`���) from `CMath`, for the following reasons.

* wanna keep CMath minimal. `CMath#sin` should be superset of `CMath#sin!` or `Math#sin` is enough.

* [rdoc for CMath seems broken](http://ruby-doc.org/stdlib-2.2.1/libdoc/cmath/rdoc/CMath.html), `CMath#sin!` isn't alias for `CMath#sin`

thanks, gogo.

---Files--------------------------------
remove_methods_from_cmath.patch (7.89 KB)
deprecate_CMath_methods.patch (6.4 KB)


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