ruby-changes:34233
From: ko1 <ko1@a...>
Date: Mon, 2 Jun 2014 17:18:09 +0900 (JST)
Subject: [ruby-changes:34233] ko1:r46314 (trunk): * vm.c (ruby_vm_destruct): remove useless call of
ko1 2014-06-02 17:17:55 +0900 (Mon, 02 Jun 2014) New Revision: 46314 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=46314 Log: * vm.c (ruby_vm_destruct): remove useless call of rb_gc_force_recycle(). At this line, a VM object is already freed (is changed to T_NONE) by rb_gc_call_finalizer_at_exit(). Modified files: trunk/ChangeLog trunk/vm.c Index: ChangeLog =================================================================== --- ChangeLog (revision 46313) +++ ChangeLog (revision 46314) @@ -1,3 +1,11 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1 +Mon Jun 2 17:14:49 2014 Koichi Sasada <ko1@a...> + + * vm.c (ruby_vm_destruct): remove useless call of + rb_gc_force_recycle(). + + At this line, a VM object is already freed + (is changed to T_NONE) by rb_gc_call_finalizer_at_exit(). + Mon Jun 2 15:50:24 2014 Koichi Sasada <ko1@a...> * eval.c (rb_using_refinement): add write-barriers for Index: vm.c =================================================================== --- vm.c (revision 46313) +++ vm.c (revision 46314) @@ -1749,12 +1749,12 @@ int https://github.com/ruby/ruby/blob/trunk/vm.c#L1749 ruby_vm_destruct(rb_vm_t *vm) { RUBY_FREE_ENTER("vm"); + if (vm) { rb_thread_t *th = vm->main_thread; #if defined(ENABLE_VM_OBJSPACE) && ENABLE_VM_OBJSPACE struct rb_objspace *objspace = vm->objspace; #endif - rb_gc_force_recycle(vm->self); vm->main_thread = 0; if (th) { rb_fiber_reset_root_local_storage(th->self); -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/