Project

General

Profile

Actions

Feature #6626

closed

Updated by shugo (Shugo Maeda) over 12 years ago

  • Tracker changed from Bug to Feature

Updated by shugo (Shugo Maeda) over 12 years ago

  • Target version changed from 2.0.0 to 2.6

feature requestだと思うのでnext minorにしておきます。

Updated by kouji (Kouji Takao) almost 12 years ago

大変遅くなりましたが、これを取り込もうと思って作業しています。

手元の環境(Mac OS X 10.7, GNU Readline 6.2)での動作確認は終わりました。
あと、Mac OSに標準添付されているEditLine(libedit)での動作を確認したらcommitしようかなと思っていたのですが、
このパッチには、string.cで定義されているrb_str_subpos関数を非公開から公開にする修正が含まれています。これを取り込んでもいいものなのか私では判断できませんでした。rb_str_subpos関数と同様の処理をext/readline/readline.cで実装すればrb_str_subpos関数を非公開のままにできると思って、rb_str_subpos関数を見てみましたが、なかなか長い処理なのですね...

すみませんが、どなたかrb_str_subpos関数を非公開から公開に変更してもよいかどうか判断していただけないでしょうか?
修正したい理由は、Readlineモジュールに対して本チケットで提案されている機能を実現するために、Readlineモジュールからrb_str_subpos関数を利用したいためです。利用できない場合は、rb_str_subpos関数と同様の処理をReadlineモジュールにも実装することになり、コードの重複がうまれ、メンテナンス性が悪くなると考えています。

Updated by kouji (Kouji Takao) almost 12 years ago

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

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


  • ext/readline/readline.c (Init_readline): added
    Readline.delete_text. [ruby-dev:45789] [Feature ]

  • ext/readline/extconf.rb: check for rl_delete_text() in Readline library.

    Thanks, Nobuyoshi Nakada, for the .

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0