From a5a777dc0aa1a198db307a1f9bb3668e4d3c76d5 Mon Sep 17 00:00:00 2001
From: normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>
Date: Fri, 17 Mar 2017 19:59:56 +0000
Subject: remove branches in dmark and dfree GC callbacks

dmark and dfree callbacks are never called in gc.c for NULL
DATA_PTR values, not even for zombie objects.

* compile.c (ibf_loader_mark): remove branch for pointer validity
* compile.c (ibf_loader_free): ditto
* cont.c (cont_free): ditto
* cont.c (fiber_free): ditto
* dir.c (dir_free): ditto
* ext/stringio/stringio.c (strio_mark): ditto
* proc.c (binding_free): ditto
* thread_sync.c (mutex_free): ditto
* vm.c (thread_free): ditto

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@58006 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
---
 proc.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

(limited to 'proc.c')

diff --git a/proc.c b/proc.c
index dd571f7ba8..58d33b199a 100644
--- a/proc.c
+++ b/proc.c
@@ -265,12 +265,10 @@ rb_proc_lambda_p(VALUE procval)
 static void
 binding_free(void *ptr)
 {
-    rb_binding_t *bind;
     RUBY_FREE_ENTER("binding");
-    if (ptr) {
-	bind = ptr;
-	ruby_xfree(bind);
-    }
+
+    ruby_xfree(ptr);
+
     RUBY_FREE_LEAVE("binding");
 }
 
-- 
cgit v1.2.3