ruby-changes:35814
From: normal <ko1@a...>
Date: Mon, 13 Oct 2014 06:53:26 +0900 (JST)
Subject: [ruby-changes:35814] normal:r47896 (trunk): array.c: swap volatile for RB_GC_GUARD
normal 2014-10-13 06:53:01 +0900 (Mon, 13 Oct 2014) New Revision: 47896 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=47896 Log: array.c: swap volatile for RB_GC_GUARD * array.c (ary_recycle_hash): add RB_GC_GUARD (rb_ary_diff): remove volatile [Bug #10369] The RB_GC_GUARD seems needed for most of the ary_recycle_hash functions anyways. Modified files: trunk/ChangeLog trunk/array.c Index: array.c =================================================================== --- array.c (revision 47895) +++ array.c (revision 47896) @@ -3986,6 +3986,7 @@ ary_recycle_hash(VALUE hash) https://github.com/ruby/ruby/blob/trunk/array.c#L3986 RHASH(hash)->ntbl = 0; st_free_table(tbl); } + RB_GC_GUARD(hash); } /* @@ -4009,7 +4010,7 @@ static VALUE https://github.com/ruby/ruby/blob/trunk/array.c#L4010 rb_ary_diff(VALUE ary1, VALUE ary2) { VALUE ary3; - volatile VALUE hash; + VALUE hash; long i; hash = ary_make_hash(to_ary(ary2)); Index: ChangeLog =================================================================== --- ChangeLog (revision 47895) +++ ChangeLog (revision 47896) @@ -1,3 +1,9 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1 +Mon Oct 13 06:52:09 2014 Eric Wong <e@8...> + + * array.c (ary_recycle_hash): add RB_GC_GUARD + (rb_ary_diff): remove volatile + [Bug #10369] + Mon Oct 13 03:20:23 2014 Zachary Scott <e@z...> * ext/date/date_core.c: [DOC] Clean up whitespace, examples, and typos -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/