From a0908cb41356f4384a3e9a199c14490a452080ef Mon Sep 17 00:00:00 2001 From: normal Date: Wed, 15 Jul 2015 08:29:22 +0000 Subject: remove redundant NULL check in mark functions gc.c (gc_mark_children)only calls mark_func if the T_DATA ptr is non-NULL, so avoid redundantly checking for that in each mark function. * iseq.c (iseq_mark): remove check for data pointer * proc.c (binding_mark): ditto * vm.c (rb_thread_mark): ditto * vm_trace.c (tp_mark): ditto git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@51252 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- vm_trace.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'vm_trace.c') diff --git a/vm_trace.c b/vm_trace.c index cec0deecb0..b46437038c 100644 --- a/vm_trace.c +++ b/vm_trace.c @@ -653,11 +653,9 @@ typedef struct rb_tp_struct { static void tp_mark(void *ptr) { - if (ptr) { - rb_tp_t *tp = (rb_tp_t *)ptr; - rb_gc_mark(tp->proc); - if (tp->target_th) rb_gc_mark(tp->target_th->self); - } + rb_tp_t *tp = ptr; + rb_gc_mark(tp->proc); + if (tp->target_th) rb_gc_mark(tp->target_th->self); } static size_t -- cgit v1.2.3