1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

gc.c: speed up RGENGC_CHECK_MODE by removing unncessary check

Browse Source 
* 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
This commit is contained in:
tmm1 2013-12-10 06:58:48 +00:00
parent 1b29c00f00
commit 40d7c9c18b
2 changed files with 11 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
ChangeLog
View File

@ -1,3 +1,10 @@
Tue Dec 10 15:56:48 2013 Aman Gupta <ruby@tmm1.net>
* 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
Tue Dec 10 15:46:03 2013 Koichi Sasada <ko1@atdot.net>
* gc.c (allrefs_add): push obj only if allrefs table doesn't have

14
gc.c
View File

@ -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 *