Project

General

Profile

Actions

Feature #10169

closed

It might be better to make Mathn class deprecated

Added by gogotanaka (Kazuki Tanaka) over 10 years ago. Updated over 7 years ago.

Status:
Closed
Target version:
[ruby-core:64553]

Description

#English

As I mentioned in https://bugs.ruby-lang.org/issues/10086#change-48156,

  1. Now that we are able to hundle complex exponents without mathn.

  2. And overriding \ with quo has too much side effect. I suppose we should not implement such a thing in ruby standard library.

  3. Requiring and integrating other standard libraries doesn't make sense.

This is why I think it might be better to make Mathn class deprecated and disabled in the future.

#日本語(すみませんまだ英語が拙く、意図をより正確にお伝えするため日本語も記載させて頂きます)

僕がこちらのissueで述べさせて頂いた通り、https://bugs.ruby-lang.org/issues/10086#change-48156

  1. mathn で実装されている冪乗演算は mathn を用いずとも現在は実現されている事

  2. mathn 内で行われている \quo でオーバーライドする事は副作用が強く、またこの種のオーバーライドが必要な場面は多くなく、必要な場合でも自ら実装する事が多いと推測される事

  3. 各種数学関連のモジュールをrequireする機能もあまり意味を持たない事

以上の理由より、とりわけRubyの標準ライブラリーで現在のmathn を提供する必要を感じません.

よってmathnを非推奨に、ゆくゆくは廃止してしまうのが良いと考えています.


Related issues 3 (0 open3 closed)

Related to Ruby master - Bug #2121: mathn/rational destroys Fixnum#/, Fixnum#quo and Bignum#/, Bignum#quoClosedkeiju (Keiju Ishitsuka)09/19/2009Actions
Related to Ruby master - Feature #13335: Gemify mathnClosedhsbt (Hiroshi SHIBATA)Actions
Related to Ruby master - Feature #13334: Removed deprecated mathn extentions.Closedhsbt (Hiroshi SHIBATA)Actions

Updated by matz (Yukihiro Matsumoto) over 10 years ago

Approved. Organize transition path.

Matz.

Updated by hsbt (Hiroshi SHIBATA) over 10 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

Applied in changeset r47282.


Updated by hsbt (Hiroshi SHIBATA) over 10 years ago

  • Status changed from Closed to Open

Updated by hsbt (Hiroshi SHIBATA) over 10 years ago

  • Related to Bug #2121: mathn/rational destroys Fixnum#/, Fixnum#quo and Bignum#/, Bignum#quo added

Updated by gogotanaka (Kazuki Tanaka) over 10 years ago

@hiroshi (Hiroshi MORIYAMA) SHIBATA

I really appreciate you doing what I should do.

Updated by gogotanaka (Kazuki Tanaka) about 10 years ago

It might be annoying question, it's still open?
If so, please let me know why, otherwise could you make it closed?
I just wanna make sure my tickets remains or not.

I'm sorry to your extra trouble : (

Actions #7

Updated by hsbt (Hiroshi SHIBATA) over 7 years ago

Updated by hsbt (Hiroshi SHIBATA) over 7 years ago

  • Status changed from Open to Assigned
  • Assignee set to hsbt (Hiroshi SHIBATA)
  • Target version set to 2.5
Actions #9

Updated by hsbt (Hiroshi SHIBATA) over 7 years ago

  • Related to Feature #13334: Removed deprecated mathn extentions. added

Updated by hsbt (Hiroshi SHIBATA) over 7 years ago

Note: We need to remove Math mode from irb before unbundling mathn. So it uses mathn library.

After that, We can remove mathn from ruby core. and ship rubygems.

I will do it.

Actions #11

Updated by hsbt (Hiroshi SHIBATA) over 7 years ago

  • Status changed from Assigned to Closed

Applied in changeset trunk|r58432.


Removed mathn.rb from stdlib. It's deprecated from Ruby 2.2.

[Feature #10169][[ruby-core:64553]]

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0