diff options
author | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-07-15 09:32:08 +0000 |
---|---|---|
committer | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-07-15 09:32:08 +0000 |
commit | dab4d077d12f4140007aa9c4299d9bdfba0e1fc3 (patch) | |
tree | 02c1fb58a1c5104c86bff3eb09a7e7febf954b21 | |
parent | a0908cb41356f4384a3e9a199c14490a452080ef (diff) |
remove redundant NULL checks after RUBY_VM_IFUNC_P
* proc.c (proc_mark): remove redundant check
* vm.c (env_mark): ditto
This doesn't change object code size, but the unstripped
executable is smaller and the code less confusing.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@51253 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | proc.c | 2 | ||||
-rw-r--r-- | vm.c | 2 |
3 files changed, 7 insertions, 2 deletions
@@ -1,3 +1,8 @@ +Wed Jul 15 18:31:18 2015 Eric Wong <e@80x24.org> + + * proc.c (proc_mark): remove redundant check + * vm.c (env_mark): ditto + Wed Jul 15 17:27:40 2015 Eric Wong <e@80x24.org> * iseq.c (iseq_mark): remove check for data pointer @@ -50,7 +50,7 @@ proc_mark(void *ptr) RUBY_MARK_UNLESS_NULL(proc->block.self); RUBY_MARK_UNLESS_NULL(rb_vm_proc_envval(proc)); if (proc->block.iseq && RUBY_VM_IFUNC_P(proc->block.iseq)) { - RUBY_MARK_UNLESS_NULL((VALUE)(proc->block.iseq)); + rb_gc_mark((VALUE)(proc->block.iseq)); } RUBY_MARK_LEAVE("proc"); } @@ -422,7 +422,7 @@ env_mark(void * const ptr) if (env->block.iseq) { if (RUBY_VM_IFUNC_P(env->block.iseq)) { - RUBY_MARK_UNLESS_NULL((VALUE)env->block.iseq); + rb_gc_mark((VALUE)env->block.iseq); } else { RUBY_MARK_UNLESS_NULL(env->block.iseq->self); |