https://bugs.ruby-lang.org/
https://bugs.ruby-lang.org/favicon.ico?1711330511
2011-08-03T12:24:45Z
Ruby Issue Tracking System
Ruby master - Bug #5152: TestDateNew#test_civil が GC.stress = true 下で Failure
https://bugs.ruby-lang.org/issues/5152?journal_id=20028
2011-08-03T12:24:45Z
nagachika (Tomoyuki Chikanaga)
nagachika00@gmail.com
<ul></ul><p>すみません、ソースコードを良く読むと RB_GC_GUARD() を挿入しないといけないのはさっきのところじゃなくて関数の最後のほうに挿入する必要がありそうでした。</p>
<p>また parse_ddd_cb() にも GC 保護が必要な変数がありそうでした。</p>
<p>@@ -1323,6 +1324,7 @@ parse_ddd_cb(VALUE m, VALUE hash)<br>
}<br>
break;<br>
}</p>
<ul>
<li>RB_GC_GUARD(s2);<br>
if (!NIL_P(s3)) {<br>
cs3 = RSTRING_PTR(s3);<br>
l3 = RSTRING_LEN(s3);<br>
@@ -1354,6 +1356,7 @@ parse_ddd_cb(VALUE m, VALUE hash)<br>
}<br>
}<br>
}</li>
<li>RB_GC_GUARD(s3);<br>
if (!NIL_P(s4)) {<br>
l4 = RSTRING_LEN(s4);</li>
</ul>
<p>@@ -1392,6 +1395,7 @@ parse_ddd_cb(VALUE m, VALUE hash)<br>
set_hash("offset", date_zone_to_diff(rb_str_new2(s1)));<br>
}<br>
}</p>
<ul>
<li>
<p>RB_GC_GUARD(s5);</p>
<p>return 1;<br>
}</p>
</li>
</ul>
Ruby master - Bug #5152: TestDateNew#test_civil が GC.stress = true 下で Failure
https://bugs.ruby-lang.org/issues/5152?journal_id=20137
2011-08-07T17:24:42Z
tadf (tadayoshi funaba)
<ul><li><strong>ruby -v</strong> changed from <i>ruby 1.9.4dev (2011-08-03 trunk 32823) [x86_64-linux]</i> to <i>-</i></li></ul><p>とりあえずコミットしておいて下さい。</p>
Ruby master - Bug #5152: TestDateNew#test_civil が GC.stress = true 下で Failure
https://bugs.ruby-lang.org/issues/5152?journal_id=20186
2011-08-09T22:47:21Z
nagachika (Tomoyuki Chikanaga)
nagachika00@gmail.com
<ul></ul><p>了解しました。<br>
今週は追加の確認ができそうにないので、とりあえず現在わかっているところを入れてしまいます。</p>
Ruby master - Bug #5152: TestDateNew#test_civil が GC.stress = true 下で Failure
https://bugs.ruby-lang.org/issues/5152?journal_id=20187
2011-08-09T23:00:50Z
nagachika (Tomoyuki Chikanaga)
nagachika00@gmail.com
<ul><li><strong>Status</strong> changed from <i>Open</i> to <i>Closed</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>This issue was solved with changeset r32896.<br>
Tomoyuki, thank you for reporting this issue.<br>
Your contribution to Ruby is greatly appreciated.<br>
May Ruby be with you.</p>
<hr>
<ul>
<li>
<p>ext/date/date_parse.c (date_zone_to_diff): add RB_GC_GUARD.<br>
<a href="/issues/5152">[ruby-dev:44337]</a> [Bug <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: TestDateNew#test_civil が GC.stress = true 下で Failure (Closed)" href="https://bugs.ruby-lang.org/issues/5152">#5152</a>]</p>
</li>
<li>
<p>ext/date/data_parse.c (parse_ddd_cb): ditto.</p>
</li>
</ul>