dubstep
December 21, 2011, 10:41am
1
Issue #5785 has been reported by Tomoyuki C…
Feature #5785: Readline に pre_input_hook, insert_text, redisplay を追加
Author: Tomoyuki C.
Status: Open
Priority: Normal
Assignee: Kouji T.
Category: ext
Target version:
拡張ライブラリ readline で補完時の処理をカスタマイズするため rl_pre_input_hook, rl_insert_text,
rl_redisplay をサポートして欲しいです。
叩き台としてパッチを添付します。
Issue #5785 has been updated by Tomoyuki C…
File readline_pre_input_hook2.patch added
trunk の変更でパッチがあたらなくなっていたので udpate したものを再度添付します。
Feature #5785: Readline に pre_input_hook, insert_text, redisplay を追加
Author: Tomoyuki C.
Status: Open
Priority: Normal
Assignee: Kouji T.
Category: ext
Target version:
拡張ライブラリ readline で補完時の処理をカスタマイズするため rl_pre_input_hook, rl_insert_text,
rl_redisplay をサポートして欲しいです。
叩き台としてパッチを添付します。
Issue #5785 has been updated by kouji (Kouji T.).
Status changed from Closed to Rejected
Feature #5785: Readline に pre_input_hook, insert_text, redisplay を追加
Author: nagachika (Tomoyuki C.)
Status: Rejected
Priority: Normal
Assignee: kouji (Kouji T.)
Category: ext
Target version:
拡張ライブラリ readline で補完時の処理をカスタマイズするため rl_pre_input_hook, rl_insert_text,
rl_redisplay をサポートして欲しいです。
叩き台としてパッチを添付します。
Issue #5785 has been updated by kouji (Kouji T.).
Status changed from Assigned to Closed
Sorry, late reply.
I’m going to implement Readline.pre_input_hook, Readline.insert_text and
Readline.redisplay. These are wrapper for rl_pre_input_hook,
rl_insert_text and rl_redisplay.
So I think you will be able to implement a proc which you want.
So, I closed this issue.
Thanks.
Feature #5785: Readline に pre_input_hook, insert_text, redisplay を追加
Author: nagachika (Tomoyuki C.)
Status: Closed
Priority: Normal
Assignee: kouji (Kouji T.)
Category: ext
Target version:
拡張ライブラリ readline で補完時の処理をカスタマイズするため rl_pre_input_hook, rl_insert_text,
rl_redisplay をサポートして欲しいです。
叩き台としてパッチを添付します。
Issue #5785 has been updated by kouji (Kouji T.).
少し補足します。
オリジナルのパッチから以下の修正をしています。
ユニットテストのtest_modify_text_in_pre_input_hookの出力結果(stdout.read)の期待値を">
hello world\n"に修正しました。
rl_pre_insert_hookのsetterとgetterの名前はReadline.pre_insert_hookにしました。
RDocを記述しました。
以上です。
なにわともあれパッチを提供いただきありがとうございました。
何か問題がありましたら、遠慮なくレポートしてください。
Feature #5785: Readline に pre_input_hook, insert_text, redisplay を追加
Author: nagachika (Tomoyuki C.)
Status: Closed
Priority: Normal
Assignee: kouji (Kouji T.)
Category: ext
Target version: 2.0.0
拡張ライブラリ readline で補完時の処理をカスタマイズするため rl_pre_input_hook, rl_insert_text,
rl_redisplay をサポートして欲しいです。
叩き台としてパッチを添付します。
Issue #5785 has been updated by kouji (Kouji T.).
Due date set to 05/02/2012
Status changed from Rejected to Assigned
Target version set to 2.0.0
すみません。#1823にコメントするつもりが間違ってこちらにコメントしてしまいました。
さっさと、これを実装します。
パッチをいただき、ありがとうございます。
Feature #5785: Readline に pre_input_hook, insert_text, redisplay を追加
Author: nagachika (Tomoyuki C.)
Status: Assigned
Priority: Normal
Assignee: kouji (Kouji T.)
Category: ext
Target version: 2.0.0
拡張ライブラリ readline で補完時の処理をカスタマイズするため rl_pre_input_hook, rl_insert_text,
rl_redisplay をサポートして欲しいです。
叩き台としてパッチを添付します。
Issue #5785 has been updated by naruse (Yui NARUSE).
Status changed from Closed to Assigned
kouji (Kouji T.) wrote:
This issue was solved with changeset r35513.
Tomoyuki, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.
ext/readline/readline.c (Readline.pre_input_hook)
(Readline.insert_text, Readline.redisplay): new function. An
original patch was created by nagachika. [Feature #5785 ]
このコミット以降、boron にて以下のようにテストに失敗しています。
boron (Debian squeeze 6.0.4) は Readline 4.3 を使っているのですが、これだと
naruse@boron:~/ruby$ ./ruby -rreadline
-e’Readline.input=open(“version.h”);Readline.readline("> ")’
のようにエコーバックされず、6.1 とかだと、
% ./ruby -rreadline
-e’Readline.input=open(“/home/naruse/ruby/version.h”);Readline.readline(">
")’
#define RUBY_VERSION “2.0.0”
と、入力が表示されるようです。
Feature #5785: Readline に pre_input_hook, insert_text, redisplay を追加
Author: nagachika (Tomoyuki C.)
Status: Assigned
Priority: Normal
Assignee: kouji (Kouji T.)
Category: ext
Target version: 2.0.0
拡張ライブラリ readline で補完時の処理をカスタマイズするため rl_pre_input_hook, rl_insert_text,
rl_redisplay をサポートして欲しいです。
叩き台としてパッチを添付します。