summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authornobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2010-08-25 09:00:17 +0000
committernobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2010-08-25 09:00:17 +0000
commitae6a9009dcb5e475ef51a3448a613e7f4428855d (patch)
tree4b4e9f062ae6ea3fb6d6b3749fdc34205952a361
parent569d8219292fefe205979fd8ea951619316d4f72 (diff)
* array.c (rb_ary_{shuffle_bang,sample}): use Random class object.
* random.c (try_get_rnd): use default_rand for Random as same as singleton methods. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@29093 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r--ChangeLog7
-rw-r--r--array.c4
-rw-r--r--random.c16
3 files changed, 20 insertions, 7 deletions
diff --git a/ChangeLog b/ChangeLog
index 79cd853eea..dcddf95751 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,4 +1,9 @@
-Wed Aug 25 17:56:40 2010 Nobuyoshi Nakada <nobu@ruby-lang.org>
+Wed Aug 25 17:59:50 2010 Nobuyoshi Nakada <nobu@ruby-lang.org>
+
+ * array.c (rb_ary_{shuffle_bang,sample}): use Random class object.
+
+ * random.c (try_get_rnd): use default_rand for Random as same as
+ singleton methods.
* array.c (rb_ary_sample): use frozen shared array to get rid of
reallocation.
diff --git a/array.c b/array.c
index b506b086d4..9e15e9aa58 100644
--- a/array.c
+++ b/array.c
@@ -3748,7 +3748,7 @@ static VALUE sym_random;
static VALUE
rb_ary_shuffle_bang(int argc, VALUE *argv, VALUE ary)
{
- VALUE *ptr, opts, randgen = Qnil;
+ VALUE *ptr, opts, randgen = rb_cRandom;
long i = RARRAY_LEN(ary);
if (OPTHASH_GIVEN_P(opts)) {
@@ -3811,7 +3811,7 @@ static VALUE
rb_ary_sample(int argc, VALUE *argv, VALUE ary)
{
VALUE nv, result, *ptr;
- VALUE opts, randgen = Qnil;
+ VALUE opts, randgen = rb_cRandom;
long n, len, i, j, k, idx[10];
len = RARRAY_LEN(ary);
diff --git a/random.c b/random.c
index a89fcb8ee9..a822a23792 100644
--- a/random.c
+++ b/random.c
@@ -229,15 +229,20 @@ static rb_random_t default_rand;
static VALUE rand_init(struct MT *mt, VALUE vseed);
static VALUE random_seed(void);
-static struct MT *
-default_mt(void)
+static rb_random_t *
+rand_start(rb_random_t *r)
{
- rb_random_t *r = &default_rand;
struct MT *mt = &r->mt;
if (!genrand_initialized(mt)) {
r->seed = rand_init(mt, random_seed());
}
- return mt;
+ return r;
+}
+
+static struct MT *
+default_mt(void)
+{
+ return &rand_start(&default_rand)->mt;
}
unsigned int
@@ -363,6 +368,9 @@ get_rnd(VALUE obj)
static rb_random_t *
try_get_rnd(VALUE obj)
{
+ if (obj == rb_cRandom) {
+ return rand_start(&default_rand);
+ }
if (!rb_typeddata_is_kind_of(obj, &random_data_type)) return NULL;
return DATA_PTR(obj);
}