diff options
author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2007-05-13 16:54:58 +0000 |
---|---|---|
committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2007-05-13 16:54:58 +0000 |
commit | aa77090aefa957855e69f83bbae43cea63101460 (patch) | |
tree | 07688caa9349687ed7aa7e8485d3c14546bb7e38 /array.c | |
parent | 2216212554d8b540fc55c63b9b382509e89c2558 (diff) | |
download | ruby-aa77090aefa957855e69f83bbae43cea63101460.tar.gz |
* array.c (rb_ary_choice): should return nil when the array is
empty.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@12276 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'array.c')
-rw-r--r-- | array.c | 6 |
1 files changed, 4 insertions, 2 deletions
@@ -2952,9 +2952,11 @@ rb_ary_shuffle(VALUE ary) static VALUE rb_ary_choice(VALUE ary) { - long i = RARRAY_LEN(ary); - long j = genrand_real()*i; + long i, j; + i = RARRAY_LEN(ary); + if (i == 0) return Qnil; + j = genrand_real()*i; return RARRAY_PTR(ary)[j]; } |