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/