diff options
author | Aaron Patterson <tenderlove@ruby-lang.org> | 2019-04-22 17:14:36 -0700 |
---|---|---|
committer | Aaron Patterson <tenderlove@ruby-lang.org> | 2019-04-22 20:08:01 -0700 |
commit | ea520ca9273699fc1c77a71bbeba4b6e06ccfc6c (patch) | |
tree | 835783d0d3a7d09ff0f6064056a06e70178928e7 /gc.c | |
parent | 660c0716595a8fc942d12317a5c55c038b830e94 (diff) | |
download | ruby-ea520ca9273699fc1c77a71bbeba4b6e06ccfc6c.tar.gz |
Prevent rb_define_(class|module) classes from moving
Before this commit, classes and modules would be registered with the
VM's `defined_module_hash`. The key was the ID of the class, but that
meant that it was possible for hash collisions to occur. The compactor
doesn't allow classes in the `defined_module_hash` to move, but if there
is a conflict, then it's possible a class would be removed from the hash
and not get pined.
This commit changes the key / value of the hash just to be the class
itself, thus preventing movement.
Diffstat (limited to 'gc.c')
-rw-r--r-- | gc.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -8159,7 +8159,7 @@ gc_verify_compaction_references(VALUE mod) heap_eden->using_page = NULL; gc_verify_internal_consistency(mod); - rb_gc_enable(); + /* GC after compaction to eliminate T_MOVED */ rb_gc(); |