diff options
| -rw-r--r-- | ChangeLog | 6 | ||||
| -rw-r--r-- | test/ruby/test_exception.rb | 8 |
2 files changed, 11 insertions, 3 deletions
@@ -1,3 +1,9 @@ +Tue Sep 13 18:37:08 2016 Koichi Sasada <ko1@atdot.net> + + * test/ruby/test_exception.rb: fix thread issues. + * use Queue instead of a local variable for synchronization. + * join created thread to soleve leaking threads warning. + Tue Sep 13 16:07:26 2016 Kazuki Yamaguchi <k@rhe.jp> * string.c (rb_str_set_len): The buffer overflow check is wrong. The diff --git a/test/ruby/test_exception.rb b/test/ruby/test_exception.rb index 03c0c819f5..10489dd5ba 100644 --- a/test/ruby/test_exception.rb +++ b/test/ruby/test_exception.rb @@ -723,9 +723,9 @@ end.join bug12741 = '[ruby-core:77222] [Bug #12741]' x = Thread.current - a = false + q = Queue.new y = Thread.start do - Thread.pass until a + q.pop begin raise "caller's cause" rescue @@ -737,9 +737,11 @@ end.join raise bug12741 rescue e = assert_raise_with_message(RuntimeError, "stop") do - a = true + q.push(true) sleep 1 end + ensure + y.join end assert_equal("caller's cause", e.cause.message) end |