Project

General

Profile

Actions

Bug #11736

closed

Float(arg) not consistent with documentation

Added by bwheeler96 (Brian Wheeler) over 8 years ago. Updated over 7 years ago.

Status:
Closed
Target version:
-
ruby -v:
ruby 2.1.5p273
[ruby-core:71661]

Description

Float(arg) → float
Returns arg converted to a float. Numeric types are converted directly, the rest are converted using arg.to_f. Converting nil generates a TypeError.

Strings are NOT converted using arg.to_f

'123_INVALID'.to_f 
# => 123.0
Float('123_INVALID') 
# => ArgumentError

This could be fixed by changing object.c like so:

-       return DBL2NUM(rb_str_to_dbl(val, TRUE));
+       return DBL2NUM(rb_str_to_dbl(val, FALSE));

But it appears that this is desired behavior as there are many specs to ensure this strict validity checking. Please update the documentation to reflect this.

Thanks,

Brian


Files


Related issues 1 (0 open1 closed)

Related to Ruby master - Bug #9574: Inconsistent behavior between Kernel#Float and to_fClosedzzak (zzak _)Actions

Updated by ko1 (Koichi Sasada) over 8 years ago

  • Description updated (diff)
  • Assignee set to zzak (zzak _)

Document is wrong.
Zack, could you fix it?

Updated by dpulliam (Dylan Pulliam) about 8 years ago

I have prepared a patch for this. Let me know what you think.

Actions #3

Updated by hsbt (Hiroshi SHIBATA) over 7 years ago

  • Status changed from Open to Closed

Applied in changeset r56421.


Updated by hsbt (Hiroshi SHIBATA) over 7 years ago

  • Assignee changed from zzak (zzak _) to hsbt (Hiroshi SHIBATA)
  • Backport changed from 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN to 2.1: UNKNOWN, 2.2: REQUIRED, 2.3: REQUIRED

Updated by usa (Usaku NAKAMURA) over 7 years ago

  • Backport changed from 2.1: UNKNOWN, 2.2: REQUIRED, 2.3: REQUIRED to 2.1: UNKNOWN, 2.2: DONE, 2.3: REQUIRED

ruby_2_2 r56501 merged revision(s) 56421,56422.

Updated by nagachika (Tomoyuki Chikanaga) over 7 years ago

  • Backport changed from 2.1: UNKNOWN, 2.2: DONE, 2.3: REQUIRED to 2.1: UNKNOWN, 2.2: DONE, 2.3: DONE

ruby_2_3 r56722 merged revision(s) 56421,56422.

Actions #7

Updated by jeremyevans0 (Jeremy Evans) over 4 years ago

  • Related to Bug #9574: Inconsistent behavior between Kernel#Float and to_f added
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0Like0