Project

General

Profile

Actions

Bug #6339

closed

IRB::Context#use_readline= の削除

Added by sho-h (Sho Hashimoto) almost 12 years ago. Updated over 11 years ago.

Status:
Closed
Target version:
-
ruby -v:
ruby 1.9.3p125 (2012-02-16 revision 34643) [x86_64-linux]
Backport:
[ruby-dev:45545]

Description

るりまを書いている最中に気づいたのですが、IRB::Context.new では use_readline と input_method によって @io への代入を分岐していますが、IRB::Context#use_readline= の実行時には @use_readline への代入とメッセージの出力しか行われないのが気になりました。

$ irb --noreadline
irb(main):001:0> conf.io
=> #<IRB::StdioInputMethod:0xf728574c @file_name="(line)", @line_no=1, @prompt="irb(main):001:0> ", @line=[nil, "conf.io\n"]>
irb(main):002:0> conf.use_readline?
=> false
irb(main):003:0> conf.use_readline = true
use readline module
=> true
irb(main):004:0> conf.io
=> #<IRB::StdioInputMethod:0xf728574c @file_name="(line)", @line_no=4, @prompt="irb(main):004:0> ", @line=[nil, "conf.io\n", "conf.use_readline?\n", "conf.use_readline = true\n", "conf.io\n"]>

最後の入力の時に Ctrl+a などタイプしても ^A と表示されるままでした。

IRB.conf[:USE_READLINE] と --readline/--noreadline は使えますので、IRB::Context#use_readline= は削除してしまうのがよいのではないかと思いました。

#なんかいっぱい登録してすみません...

Updated by mame (Yusuke Endoh) almost 12 years ago

  • Status changed from Open to Assigned
  • Assignee set to keiju (Keiju Ishitsuka)

Updated by keiju (Keiju Ishitsuka) over 11 years ago

sho-h (Sho Hashimoto) wrote:

IRB.conf[:USE_READLINE] と --readline/--noreadline は使えますので、IRB::Context#use_readline= は削除してしまうのがよいのではないかと思いました。

そうですねぇ。。。
機能するようにすることもできなくはないですが、動的に変更できるのもおかしい気がしますので、将来的には削除したいと思います。
今回は、obsolete だとメッセージを出すだけに変更することにしました。

Actions #3

Updated by keiju (Keiju Ishitsuka) over 11 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r38623.
Sho, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • lib/irb/context.rb: IRB::Context#use_readline= has been obsolate
    [Bug #6339].
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0