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 | dafbab6cff8c384c9836d3843b8dca6220ab6fbe (patch) | |
tree | 07688caa9349687ed7aa7e8485d3c14546bb7e38 /array.c | |
parent | 7902be2ea941be18091c796fa8abfeaf758830c6 (diff) | |
download | ruby-dafbab6cff8c384c9836d3843b8dca6220ab6fbe.tar.gz ruby-dafbab6cff8c384c9836d3843b8dca6220ab6fbe.tar.xz ruby-dafbab6cff8c384c9836d3843b8dca6220ab6fbe.zip |
* array.c (rb_ary_choice): should return nil when the array is
empty.
git-svn-id: http://svn.ruby-lang.org/repos/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]; } |