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

ruby-changes:32203

From: tmm1 <ko1@a...>
Date: Thu, 19 Dec 2013 03:12:02 +0900 (JST)
Subject: [ruby-changes:32203] tmm1:r44282 (trunk): gc.c: remove redundant freelist assignment

tmm1	2013-12-19 03:11:54 +0900 (Thu, 19 Dec 2013)

  New Revision: 44282

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=44282

  Log:
    gc.c: remove redundant freelist assignment
    
    * gc.c (heap_get_freeobj): remove redundant assignment. heap->freelist
      is set after the while() loop already.

  Modified files:
    trunk/ChangeLog
    trunk/gc.c
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 44281)
+++ ChangeLog	(revision 44282)
@@ -1,3 +1,8 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1
+Thu Dec 19 03:10:30 2013  Aman Gupta <ruby@t...>
+
+	* gc.c (heap_get_freeobj): remove redundant assignment. heap->freelist
+	  is set after the while() loop already.
+
 Thu Dec 19 01:54:30 2013  Koichi Sasada  <ko1@a...>
 
 	* test/runner.rb: fix commit miss on r44278.
Index: gc.c
===================================================================
--- gc.c	(revision 44281)
+++ gc.c	(revision 44282)
@@ -1240,7 +1240,7 @@ heap_get_freeobj(rb_objspace_t *objspace https://github.com/ruby/ruby/blob/trunk/gc.c#L1240
     while (UNLIKELY(p == NULL)) {
 	struct heap_page *page = heap_get_freepage(objspace, heap);
 	heap->using_page = page;
-	p = heap->freelist = page->freelist;
+	p = page->freelist;
 	page->freelist = NULL;
     }
     heap->freelist = p->as.free.next;

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

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