diff options
author | ko1 <ko1@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2014-07-08 05:48:36 +0000 |
---|---|---|
committer | ko1 <ko1@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2014-07-08 05:48:36 +0000 |
commit | fd7073f7aab5c6433a4a64a5b98c04404bbd4c14 (patch) | |
tree | d6ef6b013917d4e0da3b6cbb49f0afe618c154fc | |
parent | a560c2086ad3f9177541119ae487dcae3ffb2b87 (diff) |
* parse.y (dsymbol_alloc): set global_symbols.minor_marked to 0.
* parse.y (dsymbol_check): set RSYMBOL(sym)->fstr to 0
because we should not touch fstr after that.
* parse.y (rb_gc_free_dsymbol): skip deleting str and sym
from tables if fstr == 0.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@46752 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | parse.y | 29 |
2 files changed, 31 insertions, 8 deletions
@@ -1,3 +1,13 @@ +Tue Jul 8 14:45:17 2014 Koichi Sasada <ko1@atdot.net> + + * parse.y (dsymbol_alloc): set global_symbols.minor_marked to 0. + + * parse.y (dsymbol_check): set RSYMBOL(sym)->fstr to 0 + because we should not touch fstr after that. + + * parse.y (rb_gc_free_dsymbol): skip deleting str and sym + from tables if fstr == 0. + Mon Jul 7 14:31:52 2014 Koichi Sasada <ko1@atdot.net> * parse.y: remove global_symbols::pinned_dsym @@ -10451,6 +10451,7 @@ dsymbol_alloc(const VALUE klass, const VALUE str, rb_encoding * const enc) st_add_direct(global_symbols.str_id, (st_data_t)str, (st_data_t)dsym); st_add_direct(global_symbols.id_str, (ID)dsym, (st_data_t)str); + global_symbols.minor_marked = 0; if (RUBY_DTRACE_SYMBOL_CREATE_ENABLED()) { RUBY_DTRACE_SYMBOL_CREATE(RSTRING_PTR(RSYMBOL(dsym)->fstr), rb_sourcefile(), rb_sourceline()); @@ -10464,8 +10465,14 @@ dsymbol_check(const VALUE sym) { if (UNLIKELY(rb_objspace_garbage_object_p(sym))) { const VALUE fstr = RSYMBOL(sym)->fstr; - st_delete(global_symbols.str_id, (st_data_t *)&fstr, NULL); - st_delete(global_symbols.id_str, (st_data_t *)&sym, NULL); + RSYMBOL(sym)->fstr = 0; + + if (st_delete(global_symbols.str_id, (st_data_t *)&fstr, NULL) == 0) { + rb_bug("can't remove fstr from str_id (%s)", RSTRING_PTR(fstr)); + }; + if (st_delete(global_symbols.id_str, (st_data_t *)&sym, NULL) == 0) { + rb_bug("can't remove sym from id_sym (%s)", RSTRING_PTR(fstr)); + } return dsymbol_alloc(rb_cSymbol, fstr, rb_enc_get(fstr)); } else { @@ -10687,12 +10694,18 @@ rb_intern_str(VALUE str) void rb_gc_free_dsymbol(VALUE sym) { - st_data_t data; - data = (st_data_t)RSYMBOL(sym)->fstr; - st_delete(global_symbols.str_id, &data, 0); - data = (st_data_t)sym; - st_delete(global_symbols.id_str, &data, 0); - RSYMBOL(sym)->fstr = (VALUE)NULL; + st_data_t str_data = (st_data_t)RSYMBOL(sym)->fstr; + st_data_t sym_data = (st_data_t)sym; + + if (str_data) { + if (st_delete(global_symbols.str_id, &str_data, 0) == 0) { + rb_bug("rb_gc_free_dsymbol: %p can't remove str from str_id (%s)", (void *)sym, RSTRING_PTR(RSYMBOL(sym)->fstr)); + } + if (st_delete(global_symbols.id_str, &sym_data, 0) == 0) { + rb_bug("rb_gc_free_dsymbol: %p can't remove sym from id_str (%s)", (void *)sym, RSTRING_PTR(RSYMBOL(sym)->fstr)); + } + RSYMBOL(sym)->fstr = (VALUE)NULL; + } } /* |