[前][次][番号順一覧][スレッド一覧]

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/

[前][次][番号順一覧][スレッド一覧]