diff options
author | tmm1 <tmm1@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-12-10 06:58:48 +0000 |
---|---|---|
committer | tmm1 <tmm1@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-12-10 06:58:48 +0000 |
commit | 40d7c9c18b024f234f5fdd5c51fb843d9c229efc (patch) | |
tree | 1d712b19fedd5517d7313b21125ac612f3711239 /gc.c | |
parent | 1b29c00f00e42ade9858142b512b6eab211fb223 (diff) | |
download | ruby-40d7c9c18b024f234f5fdd5c51fb843d9c229efc.tar.gz |
gc.c: speed up RGENGC_CHECK_MODE by removing unncessary check
* gc.c (reflist_add): revert changes from r44109. it is unnecessary
after r44113
* gc.c (allrefs_i): fix whitespace
* gc.c (allrefs_roots_i): fix whitespace
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@44114 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'gc.c')
-rw-r--r-- | gc.c | 14 |
1 files changed, 4 insertions, 10 deletions
@@ -4165,21 +4165,15 @@ reflist_destruct(struct reflist *refs) xfree(refs); } -static int +static void reflist_add(struct reflist *refs, VALUE obj) { - int i = 0; if (refs->pos == refs->size) { refs->size *= 2; SIZED_REALLOC_N(refs->list, VALUE, refs->size, refs->size/2); } - for (i=0; i<refs->pos; i++) - if (refs->list[i] == obj) - return 0; /* already exists */ - refs->list[refs->pos++] = obj; - return 1; /* added */ } static void @@ -4248,7 +4242,7 @@ allrefs_i(VALUE obj, void *ptr) if (allrefs_add(data, obj)) { push_mark_stack(&data->objspace->mark_stack, obj); -} + } } static void @@ -4259,8 +4253,8 @@ allrefs_roots_i(VALUE obj, void *ptr) data->root_obj = MAKE_ROOTSIG(data->category); if (allrefs_add(data, obj)) { - push_mark_stack(&data->objspace->mark_stack, obj); -} + push_mark_stack(&data->objspace->mark_stack, obj); + } } static st_table * |