diff options
| author | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-06-30 21:07:18 +0000 |
|---|---|---|
| committer | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-06-30 21:07:18 +0000 |
| commit | b49075cdf15a733cf7a9b1f9ce9d06c40828db3d (patch) | |
| tree | ffe20a1e7d680301a086083bd3ee77b081eba29d /ChangeLog | |
| parent | c7548028a20c555b692c07c37b3ffd2d416bb434 (diff) | |
move RB_GC_GUARD responsibility to rb_add_method_iseq
This simplifies all the callers and makes code easier to use
and review. I was confused about the need for RB_GC_GUARD
in define_{aset,aref}_method of struct.c without reading
rb_add_method_iseq.
Likewise, do the same for rb_iseq_clone, where the GC guard
only seems neccesary iff RGenGC is disabled.
* vm_method.c (rb_add_method_iseq): add RB_GC_GUARD
* class.c (clone_method): remove RB_GC_GUARD
* struct.c (define_aref_method): ditto
(define_aset_method): ditto
* vm.c (vm_define_method):
* iseq.c (rb_iseq_clone): add RB_GC_GUARD
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@51079 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'ChangeLog')
| -rw-r--r-- | ChangeLog | 9 |
1 files changed, 9 insertions, 0 deletions
@@ -1,3 +1,12 @@ +Wed Jul 1 05:57:03 2015 Eric Wong <e@80x24.org> + + * vm_method.c (rb_add_method_iseq): add RB_GC_GUARD + * class.c (clone_method): remove RB_GC_GUARD + * struct.c (define_aref_method): ditto + (define_aset_method): ditto + * vm.c (vm_define_method): + * iseq.c (rb_iseq_clone): add RB_GC_GUARD + Wed Jul 1 05:43:58 2015 Eric Wong <e@80x24.org> * struct.c (AREF_HASH_THRESHOLD): new macro |