Project

General

Profile

Bug #5515

Integer#quo(aFloat) の結果が Rational になる

Added by tadf (tadayoshi funaba) almost 6 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Target version:
ruby -v:
ruby 2.0.0dev (2011-10-30 trunk 33566) [i686-linux]
Backport:
[ruby-dev:44710]

Description

Integer#quo(aFloat) の結果が Rational になる。

5.quo(2.0) #=> (5/2)

1.9.2 では Float になります。


Related issues

Related to Ruby trunk - Bug #8894: Fixnum#quo returns wrong result when given a floatClosed

Associated revisions

Revision 41099
Added by mrkn (Kenta Murata) about 4 years ago

  • numeric.c (num_quo): should return a Float for a Float argument.
    [Bug #5515]

  • test/ruby/test_fixnum.rb: Add an assertion for the above change.

  • test/ruby/test_bignum.rb: ditto.

Revision 41099
Added by mrkn (Kenta Murata) about 4 years ago

  • numeric.c (num_quo): should return a Float for a Float argument.
    [Bug #5515]

  • test/ruby/test_fixnum.rb: Add an assertion for the above change.

  • test/ruby/test_bignum.rb: ditto.

Revision 41099
Added by mrkn (Kenta Murata) about 4 years ago

  • numeric.c (num_quo): should return a Float for a Float argument.
    [Bug #5515]

  • test/ruby/test_fixnum.rb: Add an assertion for the above change.

  • test/ruby/test_bignum.rb: ditto.

Revision 41099
Added by mrkn (Kenta Murata) about 4 years ago

  • numeric.c (num_quo): should return a Float for a Float argument.
    [Bug #5515]

  • test/ruby/test_fixnum.rb: Add an assertion for the above change.

  • test/ruby/test_bignum.rb: ditto.

History

#1 [ruby-dev:45328] Updated by ko1 (Koichi Sasada) over 5 years ago

  • Category set to core
  • Assignee set to mrkn (Kenta Murata)

#2 Updated by shyouhei (Shyouhei Urabe) over 5 years ago

  • Status changed from Open to Assigned

#3 Updated by mrkn (Kenta Murata) about 4 years ago

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

This issue was solved with changeset r41099.
tadayoshi, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • numeric.c (num_quo): should return a Float for a Float argument.
    [Bug #5515]

  • test/ruby/test_fixnum.rb: Add an assertion for the above change.

  • test/ruby/test_bignum.rb: ditto.

#4 [ruby-dev:47715] Updated by tadf (tadayoshi funaba) almost 4 years ago

  • Status changed from Closed to Open
  • Priority changed from Normal to 5
  • Target version deleted (Next Major)

直ってないみたい。
そもそも r28844 で間違った事してて、trunk でも表面的に対処してるだけだった。

#5 [ruby-dev:47716] Updated by tadf (tadayoshi funaba) almost 4 years ago

$ ruby -v -e 'p 5.quo(2.0)'
ruby 2.0.0p317 (2013-09-15 revision 42947) i686-linux

#6 [ruby-dev:47717] Updated by naruse (Yui NARUSE) almost 4 years ago

  • Status changed from Open to Assigned
  • Target version set to 2.1.0

#7 [ruby-dev:47864] Updated by nobu (Nobuyoshi Nakada) over 3 years ago

  • Status changed from Assigned to Closed

Also available in: Atom PDF