From: Yusuke ENDOH <mame@...>
Date: 2008-08-21T22:06:12+09:00
Subject: [ruby-dev:35902] Re: [Bug:1.8][Bug:1.8.7] irb fg

遠藤です。

2008/06/13 12:04 石塚圭樹 <keiju@ishitsuka.com>:
>>> fgに引数を指定しなかったりjobsにない数値を指定すると
>>> 「Maybe IRB bug!!」になります。
>>
>>1.8, 1.8.7, 1.8.6, 1.8.5でも同様です。
>
> irb関連コマンドはエラー処理のことあまり考えてないですねぇ...
> ちょっと考えてみます.

とりあえずこの問題のパッチだけ書いてみました。
どうでしょうか。


Index: lib/irb/ext/multi-irb.rb
===================================================================
--- lib/irb/ext/multi-irb.rb	(revision 18754)
+++ lib/irb/ext/multi-irb.rb	(working copy)
@@ -69,7 +69,7 @@
     end

     def search(key)
-      case key
+      job = case key
       when Integer
 	@jobs[key]
       when Irb
@@ -77,10 +77,10 @@
       when Thread
 	@jobs.assoc(key)
       else
-	assoc = @jobs.find{|k, v| v.context.main.equal?(key)}
-	IRB.fail NoSuchJob, key if assoc.nil?
-	assoc
+	@jobs.find{|k, v| v.context.main.equal?(key)}
       end
+      IRB.fail NoSuchJob, key if job.nil?
+      job
     end

     def delete(key)
Index: lib/irb/extend-command.rb
===================================================================
--- lib/irb/extend-command.rb	(revision 18754)
+++ lib/irb/extend-command.rb	(working copy)
@@ -125,9 +125,14 @@
 	eval %[
 	  def #{cmd_name}(*opts, &b)
 	    require "#{load_file}"
+	    arity = ExtendCommand::#{cmd_class}.instance_method(:execute).arity
+	    args = (1..arity.abs).map {|i| "arg" + i.to_s }
+	    args << "*opts" if arity < 0
+	    args << "&block"
+	    args = args.join(", ")
 	    eval %[
-	      def #{cmd_name}(*opts, &b)
-		ExtendCommand::#{cmd_class}.execute(irb_context, *opts, &b)
+	      def #{cmd_name}(\#{args})
+		ExtendCommand::#{cmd_class}.execute(irb_context, \#{args})
 	      end
 	    ]
 	    send :#{cmd_name}, *opts, &b

-- 
Yusuke ENDOH <mame@tsg.ne.jp>