[ruby-dev:41043] Re: [Bug #3182] multi-irb may run parallelly

けいじゅ＠いしつかです.

In [ruby-dev:41031] the message: "[ruby-dev:41031] [Bug #3182]
multi-irb may run parallelly", on Apr/21 20:54(JST) Yusuke Endoh
writes:

>いしつかさん
>遠藤です。

>私の環境で再現はしないのですが、同じ手順を実行すると、kill したあとで、
>キー入力が不思議な挙動になりました。
>
>  irb(main):001:0> irb String
>  irb#1(String):001:0> irb Array
>  irb#2(Array):001:0> kill String
>  => #<IRB::Irb: @context=#<IRB::Context:0x84096bc>, @signal_status=:IN_EVAL, @scanner=#<RubyLex:0x84080a0>>
>  irb(main):002:0> => [String]
>  irb#2(Array):002:0> j
>  NameError: undefined local variable or method `j' for main:Object
>          from (irb):2
>          from ../ruby-trunk-local/bin/irb:12:in `<main>'
>  irb(main):003:0> obs
>  NameError: undefined local variable or method `obs' for main:Object
>          from (irb):3
>          from ../ruby-trunk-local/bin/irb:12:in `<main>'

>SEGV はおいといて、複数の irb スレッドが走りだすのは multi-irb のバグ
>だと思います。

確かに. 

>irb が終了したときに、親の irb スレッドを run するコードがありますが、
>kill された時は kill した irb スレッドがすでに走っているので、親を run
>させるべきでないと思います。
>
>以下のように、kill された irb 一覧を保持しておいて、kill された irb の
>終了時には親を起こさないようにしたところ、不思議な挙動はなくなりまし
>た。

確かにこれでもよい気がしますが, 単純に その終了するsubirbがカレントジョ
ブの時のみカレントジョブの置き換えをするようにするで十分なようです. こ
ちらのパッチを採用したいと思います. 

--- lib/irb/ext/multi-irb.rb	(リビジョン 27443)
+++ lib/irb/ext/multi-irb.rb	(作業コピー)
@@ -172,12 +172,14 @@
       ensure
 	unless system_exit
 	  @JobManager.delete(irb)
-	  if parent_thread.alive?
-	    @JobManager.current_job = @JobManager.irb(parent_thread)
-	    parent_thread.run
-	  else
-	    @JobManager.current_job = @JobManager.main_irb
-	    @JobManager.main_thread.run
+	  if @JobManager.current_job == irb
+	    if parent_thread.alive?
+	      @JobManager.current_job = @JobManager.irb(parent_thread)
+	      parent_thread.run
+	    else
+	      @JobManager.current_job = @JobManager.main_irb
+	      @JobManager.main_thread.run
+	    end
 	  end
 	end
       end

__
---------------------------------------------------->> 石塚 圭樹 <<---
---------------------------------->> e-mail: [e-mail] <<---