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/