[前][次][番号順一覧][スレッド一覧]

ruby-changes:35065

From: normal <ko1@a...>
Date: Tue, 12 Aug 2014 10:17:30 +0900 (JST)
Subject: [ruby-changes:35065] normal:r47147 (trunk): vm_method.c (release_method_definition): use rb_free_method_entry

normal	2014-08-12 10:17:17 +0900 (Tue, 12 Aug 2014)

  New Revision: 47147

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=47147

  Log:
    vm_method.c (release_method_definition): use rb_free_method_entry
    
    We are not currently avoiding recursion by calling
    release_method_definition, so use rb_free_method_entry directly
    to make changing code easier in case we change the way
    rb_method_entry_t is allocated.

  Modified files:
    trunk/ChangeLog
    trunk/vm_method.c
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 47146)
+++ ChangeLog	(revision 47147)
@@ -1,3 +1,7 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1
+Tue Aug 12 10:10:42 2014  Eric Wong  <e@8...>
+
+	* vm_method.c (release_method_definition): use rb_free_method_entry
+
 Tue Aug 12 06:16:09 2014  Masaki Suketa <masaki.suketa@n...>
 
 	* ext/win32ole/win32ole.c: seperate WIN32OLE_METHOD src from
Index: vm_method.c
===================================================================
--- vm_method.c	(revision 47146)
+++ vm_method.c	(revision 47147)
@@ -176,8 +176,7 @@ release_method_definition(rb_method_defi https://github.com/ruby/ruby/blob/trunk/vm_method.c#L176
     if (def->alias_count == 0) {
 	if (def->type == VM_METHOD_TYPE_REFINED &&
 	    def->body.orig_me) {
-	    release_method_definition(def->body.orig_me->def);
-	    xfree(def->body.orig_me);
+	    rb_free_method_entry(def->body.orig_me);
 	}
 	xfree(def);
     }

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]