diff options
| author | KOBAYASHI Shuji <[email protected]> | 2019-12-10 21:31:02 +0900 |
|---|---|---|
| committer | KOBAYASHI Shuji <[email protected]> | 2019-12-10 21:31:30 +0900 |
| commit | bf431e77b8851e87f1a65ad3bf20d7e035b31472 (patch) | |
| tree | e4ba019202bb26a8abc226438940072fc2c3322c /test | |
| parent | 543a9f84d11c2c0e96de52bd2e936467f13d26e8 (diff) | |
| download | mruby-bf431e77b8851e87f1a65ad3bf20d7e035b31472.tar.gz mruby-bf431e77b8851e87f1a65ad3bf20d7e035b31472.zip | |
Fix behavior of `String#to_i`/`Kernel#Integer` to numbers starting with `_`
#### Before this patch:
```ruby
Integer("_1") #=> 1
"_1".to_i #=> 1
```
#### After this patch (same as Ruby):
```ruby
Integer("_1") #=> ArgumentError
"_1".to_i #=> 0
```
Diffstat (limited to 'test')
| -rw-r--r-- | test/t/string.rb | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/test/t/string.rb b/test/t/string.rb index 6dbc91963..ed7e4b8f4 100644 --- a/test/t/string.rb +++ b/test/t/string.rb @@ -699,10 +699,14 @@ assert('String#to_f', '15.2.10.5.38') do assert_operator(68.0, :eql?, '68_'.to_f) assert_operator(68.0, :eql?, '68._7'.to_f) assert_operator(68.7, :eql?, '68.7_'.to_f) + assert_operator(6.0, :eql?, '6 8.7'.to_f) + assert_operator(68.0, :eql?, '68. 7'.to_f) assert_operator(0.0, :eql?, '_68'.to_f) + assert_operator(0.0, :eql?, ' _68'.to_f) assert_operator(12.34, :eql?, '1_2.3_4'.to_f) assert_operator(12.3, :eql?, '1_2.3__4'.to_f) assert_operator(0.9, :eql?, '.9'.to_f) + assert_operator(0.9, :eql?, "\t\r\n\f\v .9 \t\r\n\f\v".to_f) end if Object.const_defined?(:Float) assert('String#to_i', '15.2.10.5.39') do @@ -716,6 +720,10 @@ assert('String#to_i', '15.2.10.5.39') do assert_operator 12, :eql?, '1_2__3'.to_i assert_operator 123, :eql?, '1_2_3'.to_i assert_operator 68, :eql?, '68_'.to_i + assert_operator 0, :eql?, '_68'.to_i + assert_operator 0, :eql?, ' _68'.to_i + assert_operator 68, :eql?, "\t\r\n\f\v 68 \t\r\n\f\v".to_i + assert_operator 6, :eql?, ' 6 8 '.to_i end assert('String#to_s', '15.2.10.5.40') do |